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 13666b75ffdfc2a1ac22e0c289969648ae89e7af..8e42f5b46fcdd9e6ad1e226fb21c03514d4b8f47 100644 (file)
--- 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 1df8e192e0806eccab8cd4ec3817df85a4bb3ade..41b38b494532d5bfa73d962146b0710482d7d56e 100644 (file)
--- 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`.