]> git.ipfire.org Git - thirdparty/google/fonts.git/commitdiff
Big Shoulders Inline Text SC: correct override config source path for fontc_crater...
authorFelipe Correa da Silva Sanches <juca@members.fsf.org>
Fri, 29 May 2026 08:06:22 +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 (the source filename already matched the pinned commit).
Recipe output paths made repo-root-relative likewise.

Repo:   xotypeco/big_shoulders
Commit: 0b3d09a86862b19efae28eae0cd868f17c476b20 (unchanged; verified correct)
Config: ../Big-Shoulders... -> Big-Shoulders-Inline/sources/BigShouldersInline.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/bigshouldersinlinetextsc/config.yaml
ofl/bigshouldersinlinetextsc/upstream_info.md

index 276bb25099cf20c6c9cfd95d416f6967cea5cbb0..3cad61de235866aa1c50e0328757461c8a0da277 100644 (file)
@@ -1,8 +1,8 @@
 sources:
-    - ../Big-Shoulders-Inline/sources/BigShouldersInline.glyphs
+    - Big-Shoulders-Inline/sources/BigShouldersInline.glyphs
 recipe:
-  ../Big-Shoulders-Inline/fonts/variable/text/BigShouldersInlineText[wght].ttf:
-    - source: ../Big-Shoulders-Inline/sources/BigShouldersInline.glyphs
+  Big-Shoulders-Inline/fonts/variable/text/BigShouldersInlineText[wght].ttf:
+    - source: Big-Shoulders-Inline/sources/BigShouldersInline.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-Inline/fonts/variable/text/BigShouldersInlineTextSC[wght].ttf:
-    - source: ../Big-Shoulders-Inline/sources/BigShouldersInline.glyphs
+  Big-Shoulders-Inline/fonts/variable/text/BigShouldersInlineTextSC[wght].ttf:
+    - source: Big-Shoulders-Inline/sources/BigShouldersInline.glyphs
     - args: --filter ...  --filter FlattenComponentsFilter --filter DecomposeTransformedComponentsFilter
       operation: buildVariable
     - operation: subspace
index debd1cc8ac8c4aa837250889725974b4a2f1a04c..da548c8b1601930c04cee483939823fc23176266 100644 (file)
@@ -86,3 +86,12 @@ The METADATA.pb `files` block references `Big-Shoulders-Inline/fonts/variable/te
 ## Confidence: HIGH
 
 The repository URL and commit hash were explicitly documented in both the onboarding commit message and PR #7787 body. The commit was verified as the latest upstream commit before the merge date. The override config.yaml was present from the initial onboarding and correctly handled building the SC variant from the Glyphs source.
+
+
+## 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 (the source filename already matched the file present at the pinned commit): `Big-Shoulders-Inline/sources/BigShouldersInline.glyphs` (verified present at the pinned commit `0b3d09a`). 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.