]> git.ipfire.org Git - thirdparty/google/fonts.git/commitdiff
Big Shoulders Display: correct override config source path for fontc_crater build
authorFelipe Correa da Silva Sanches <juca@members.fsf.org>
Fri, 29 May 2026 08:06:21 +0000 (09:06 +0100)
committerFelipe Corrêa da Silva Sanches <juca@members.fsf.org>
Fri, 29 May 2026 09:48:35 +0000 (10:48 +0100)
The override config.yaml used `../`-prefixed paths that escape the repo
root when resolved by the build harness. Removed the `../` prefix and corrected the .glyphs source name to the underscored form present at the pinned commit.
Recipe output paths made repo-root-relative likewise.

Repo:   xotypeco/big_shoulders
Commit: 41153e6fe01d218e933919a1d08c8e45065bc8fe (unchanged; verified correct)
Config: ../Big-Shoulders... -> Big-Shoulders/sources/Big_Shoulders.glyphs (and recipe outputs)
Status: source path corrected (verified present at build commit; gftools-builder smoke-test passed)
Confidence: High

Assisted by an AI agent (Claude Opus 4.8)

ofl/bigshouldersdisplay/config.yaml
ofl/bigshouldersdisplay/upstream_info.md

index 7be3e862011bb29de84928fce115a0f682e65c51..7be80e8065cc57afd0e51e9ee1a00d0cd35a972e 100644 (file)
@@ -1,8 +1,8 @@
 sources:
-    - ../Big-Shoulders/sources/BigShoulders.glyphs
+    - Big-Shoulders/sources/Big_Shoulders.glyphs
 recipe:
-  ../Big-Shoulders/fonts/variable/display/BigShouldersDisplay[wght].ttf:
-    - source: ../Big-Shoulders/sources/BigShoulders.glyphs
+  Big-Shoulders/fonts/variable/display/BigShouldersDisplay[wght].ttf:
+    - source: Big-Shoulders/sources/Big_Shoulders.glyphs
     - args: --filter ...  --filter FlattenComponentsFilter --filter DecomposeTransformedComponentsFilter
       operation: buildVariable
     - operation: subspace
@@ -13,8 +13,8 @@ recipe:
     - operation: fix
     - postprocess: buildStat
   # And now the small caps family
-  ../Big-Shoulders/fonts/variable/display/BigShouldersDisplaySC[wght].ttf:
-    - source: ../Big-Shoulders/sources/BigShoulders.glyphs
+  Big-Shoulders/fonts/variable/display/BigShouldersDisplaySC[wght].ttf:
+    - source: Big-Shoulders/sources/Big_Shoulders.glyphs
     - args: --filter ...  --filter FlattenComponentsFilter --filter DecomposeTransformedComponentsFilter
       operation: buildVariable
     - operation: subspace
index d80cc59b5492a74d4292071e71d78bb13b119966..ead010d2150bc00f2f675757808be45d98d5344b 100644 (file)
@@ -78,3 +78,12 @@ Because an override config.yaml exists in google/fonts, the `config_yaml` field
 
 1. **Commit hash correction**: The recorded commit hash `41153e6` should likely be updated to `465a9c592f06d493841b35dca5d248c8142b75f8` to match the actual commit used for the v2.000 update. The file path in METADATA.pb (`Big-Shoulders/fonts/variable/display/BigShouldersDisplay[wght].ttf`) only exists at the later commit.
 2. **Relationship to parent family**: Big Shoulders Display is a derivative of the parent Big Shoulders family (same upstream repo, different optical size slice). The parent family uses commit `8ba99c9` (HEAD of master), while Big Shoulders Display uses an older commit from 2021.
+
+
+## Correction (2026-05-28) — override config source path
+
+**Model**: Claude Opus 4.8
+
+fontc_crater reported `missing source '../Big-Shoulders/sources/BigShoulders.glyphs'` for the xotypeco/big_shoulders monorepo. The override `config.yaml` used paths prefixed with `../`, which the build harness resolves relative to the repository root and therefore escape the checkout. For this family the repo-escaping `../` prefix was removed and the `.glyphs` source name was corrected to the underscored form present at the pinned commit: `Big-Shoulders/sources/Big_Shoulders.glyphs` (verified present at the pinned commit `41153e6`). The recipe output paths were made repo-root-relative likewise. The pinned commit is unchanged.
+
+A local gftools-builder smoke-test of the corrected config built the variable TTFs successfully (RC=0), confirming the path fix is sufficient.