]> git.ipfire.org Git - thirdparty/google/fonts.git/commitdiff
Fragment Mono: add override config.yaml for fontc_crater build
authorFelipe Correa da Silva Sanches <juca@members.fsf.org>
Thu, 21 May 2026 20:34:33 +0000 (21:34 +0100)
committerFelipe Corrêa da Silva Sanches <juca@members.fsf.org>
Thu, 21 May 2026 22:03:22 +0000 (23:03 +0100)
The upstream sources/config.yaml lives in sources/ and lists its source
as the bare path Fragment-Mono.glyphs, which only resolves relative to
the config's own directory. The build harness resolves source paths from
the repository root, so fontc_crater failed with "missing source
'Fragment-Mono.glyphs'".

Added an override config.yaml with the repo-root-relative source path
sources/Fragment-Mono.glyphs and removed the config_yaml field from
METADATA.pb so the override is auto-detected. The recorded commit is
correct.

Assisted by an AI agent (Claude Opus 4.7)

ofl/fragmentmono/METADATA.pb
ofl/fragmentmono/config.yaml [new file with mode: 0644]
ofl/fragmentmono/upstream_info.md

index 6b3ebc3e4208d3c1eab1e8f5e361fc2a2975cbeb..16a7a3838f9fdd57dcdf02608308fbc038b6204a 100644 (file)
@@ -42,7 +42,6 @@ source {
     dest_file: "FragmentMono-Italic.ttf"
   }
   branch: "main"
-  config_yaml: "sources/config.yaml"
 }
 stroke: "SANS_SERIF"
 classifications: "MONOSPACE"
diff --git a/ofl/fragmentmono/config.yaml b/ofl/fragmentmono/config.yaml
new file mode 100644 (file)
index 0000000..34cd448
--- /dev/null
@@ -0,0 +1,8 @@
+# Override config.yaml for ofl/fragmentmono
+# The upstream sources/config.yaml lists the source as "Fragment-Mono.glyphs",
+# a path correct only relative to the config's own sources/ directory. This
+# override makes the path repo-root-relative for the build harness.
+sources:
+  - sources/Fragment-Mono.glyphs
+familyName: Fragment Mono
+buildVariable: false
index 4abc8d69347354ee4d238811589ea4174d81aca2..465c8f01b68db9dba678b133925d1d010c7f0bac 100644 (file)
@@ -108,3 +108,21 @@ The source file has approximately 14,000 lines of diff between the onboarding co
 - **Override config**: Not needed (upstream has config.yaml)
 - **Overall status**: needs_correction
 - **Confidence**: HIGH (binary hash match confirms the correct commit)
+
+## fontc_crater Build Fix (2026-05-21)
+
+**Model**: Claude Opus 4.7
+
+### Initial state
+METADATA.pb pointed `config_yaml` at the upstream `sources/config.yaml`, whose `sources:` list is `Fragment-Mono.glyphs`. fontc_crater failed with `missing source 'Fragment-Mono.glyphs'`. (The commit-hash issue noted above has since been corrected — METADATA.pb now records the verified onboarding commit `3ff0278`.)
+
+### Investigation
+The upstream `sources/config.yaml` lives inside `sources/`, so its bare `Fragment-Mono.glyphs` entry resolves correctly only relative to the config's own directory. The build harness resolves source paths relative to the repository root, so it could not locate the `.glyphs` file. At the recorded commit `3ff0278` the source is `sources/Fragment-Mono.glyphs`.
+
+### Actions taken
+An override `config.yaml` was created in the family directory with the source path written repo-root-relative (`sources/Fragment-Mono.glyphs`). The `config_yaml` field was removed from METADATA.pb so the override is auto-detected. The recorded commit was left unchanged.
+
+Note: the sibling family Fragment Mono SC (`ofl/fragmentmonosc`) shares this upstream repo and carries an override config with the same un-prefixed `Fragment-Mono.glyphs` path — it has the identical latent defect and should receive the same correction in a follow-up.
+
+### Final state
+The override `config.yaml` references `sources/Fragment-Mono.glyphs`, which exists at the recorded commit `3ff0278`.