From: Felipe Correa da Silva Sanches <juca@members.fsf.org>
Date: Fri, 29 May 2026 08:06:21 +0000 (+0100)
Subject: Big Shoulders Text SC: correct override config source path for fontc_crater build
X-Git-Url: http://git.ipfire.org/gitweb/index.cgi?a=commitdiff_plain;h=fbbcba5083e08f4057768066d2b4a76b0ae2d52f;p=thirdparty%2Fgoogle%2Ffonts.git

Big Shoulders Text SC: correct override config source path for fontc_crater build

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/sources/BigShoulders.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)
---

diff --git a/ofl/bigshoulderstextsc/config.yaml b/ofl/bigshoulderstextsc/config.yaml
index d04ec2abb7..b90ad99361 100644
--- a/ofl/bigshoulderstextsc/config.yaml
+++ b/ofl/bigshoulderstextsc/config.yaml
@@ -1,8 +1,8 @@
 sources:
-    - ../Big-Shoulders/sources/BigShoulders.glyphs
+    - Big-Shoulders/sources/BigShoulders.glyphs
 recipe:
-  ../Big-Shoulders/fonts/variable/text/BigShouldersText[wght].ttf:
-    - source: ../Big-Shoulders/sources/BigShoulders.glyphs
+  Big-Shoulders/fonts/variable/text/BigShouldersText[wght].ttf:
+    - source: Big-Shoulders/sources/BigShoulders.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/text/BigShouldersTextSC[wght].ttf:
-    - source: ../Big-Shoulders/sources/BigShoulders.glyphs
+  Big-Shoulders/fonts/variable/text/BigShouldersTextSC[wght].ttf:
+    - source: Big-Shoulders/sources/BigShoulders.glyphs
     - args: --filter ...  --filter FlattenComponentsFilter --filter DecomposeTransformedComponentsFilter
       operation: buildVariable
     - operation: subspace
diff --git a/ofl/bigshoulderstextsc/upstream_info.md b/ofl/bigshoulderstextsc/upstream_info.md
index 07976e9960..87a882e076 100644
--- a/ofl/bigshoulderstextsc/upstream_info.md
+++ b/ofl/bigshoulderstextsc/upstream_info.md
@@ -83,3 +83,12 @@ Merged via PR #7790 on 2024-06-25.
 ## Conclusion
 
 The source metadata for Big Shoulders Text SC is **complete**. The repository URL and commit hash are correct and verified. The override config.yaml in google/fonts was created at onboarding time by Simon Cozens (PR #7790) and correctly builds the Text SC variable font from the shared Glyphs source. The `config_yaml` field is correctly omitted from METADATA.pb since the local override is auto-detected. No changes are needed.
+
+
+## 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/sources/BigShoulders.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.