From: Felipe Correa da Silva Sanches <juca@members.fsf.org>
Date: Thu, 21 May 2026 20:34:32 +0000 (+0100)
Subject: Arsenal SC: fix override config source paths for fontc_crater build
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=4258e32d2c6f911bed2f1cfe2952270dc29e4e61;p=thirdparty%2Fgoogle%2Ffonts.git

Arsenal SC: fix override config source paths for fontc_crater build

The override config.yaml listed the Glyphs sources as Arsenal.glyphs and
Arsenal-Italic.glyphs without the sources/ directory prefix. At the
recorded commit (e34db566) the sources are sources/Arsenal.glyphs and
sources/Arsenal-Italic.glyphs. fontc_crater failed with "missing source
'Arsenal-Italic.glyphs'".

Corrected the config.yaml source paths to include the sources/ prefix,
matching the regular Arsenal family's config. The recorded commit is
correct and was left unchanged.

Assisted by an AI agent (Claude Opus 4.7)
---

diff --git a/ofl/arsenalsc/config.yaml b/ofl/arsenalsc/config.yaml
index 13666b75ff..8e42f5b46f 100644
--- a/ofl/arsenalsc/config.yaml
+++ b/ofl/arsenalsc/config.yaml
@@ -1,4 +1,4 @@
 buildVariable: false
 sources:
-  - Arsenal.glyphs
-  - Arsenal-Italic.glyphs
+  - sources/Arsenal.glyphs
+  - sources/Arsenal-Italic.glyphs
diff --git a/ofl/arsenalsc/upstream_info.md b/ofl/arsenalsc/upstream_info.md
index 1df8e192e0..41b38b4945 100644
--- a/ofl/arsenalsc/upstream_info.md
+++ b/ofl/arsenalsc/upstream_info.md
@@ -49,4 +49,20 @@ The upstream repo does NOT contain pre-built `ArsenalSC-*.ttf` files. The METADA
 
 ## Conclusion
 
-All source metadata is present and verified. Repository URL, commit hash, and override config.yaml were all set during the original onboarding by Simon Cozens in PR #7771 (merged 2024-05-30 by vv-monsalve). The `files {}` mappings in METADATA.pb reference non-existent upstream paths (`fonts/TTF/ArsenalSC-*.ttf`), which appears to be a documentation artifact since the SC fonts were built, not copied from upstream. The override config.yaml source paths lack the `sources/` prefix present in the regular Arsenal config, which may warrant review. No action required for source metadata enrichment.
+All source metadata is present and verified. Repository URL, commit hash, and override config.yaml were all set during the original onboarding by Simon Cozens in PR #7771 (merged 2024-05-30 by vv-monsalve). The `files {}` mappings in METADATA.pb reference non-existent upstream paths (`fonts/TTF/ArsenalSC-*.ttf`), which appears to be a documentation artifact since the SC fonts were built, not copied from upstream. The override config.yaml source paths lack the `sources/` prefix present in the regular Arsenal config, which may warrant review.
+
+## fontc_crater Build Fix (2026-05-21)
+
+**Model**: Claude Opus 4.7
+
+### Initial state
+The override `config.yaml` listed `sources: [Arsenal.glyphs, Arsenal-Italic.glyphs]` without the `sources/` directory prefix. fontc_crater failed with `missing source 'Arsenal-Italic.glyphs'`.
+
+### Investigation
+The path discrepancy flagged in "How Config Was Resolved" above was confirmed to be an error, not intentional. At the recorded commit `e34db566` the Glyphs sources are `sources/Arsenal.glyphs` and `sources/Arsenal-Italic.glyphs`. The build harness resolves override-config source paths relative to the repository root, so the un-prefixed paths could not be located. The regular Arsenal family's override config already uses the correct `sources/`-prefixed paths.
+
+### Actions taken
+The override `config.yaml` source paths were corrected to `sources/Arsenal.glyphs` and `sources/Arsenal-Italic.glyphs`, matching the regular Arsenal family's config and the actual upstream layout. The recorded commit is correct and was left unchanged.
+
+### Final state
+The override `config.yaml` references `sources/Arsenal.glyphs` and `sources/Arsenal-Italic.glyphs`, which exist at the recorded commit `e34db566`.