From: Felipe Correa da Silva Sanches Date: Fri, 29 May 2026 08:06:22 +0000 (+0100) Subject: Big Shoulders Stencil Text SC: correct override config source path for fontc_crater... X-Git-Url: http://git.ipfire.org/gitweb/index.cgi?a=commitdiff_plain;h=1dc8dd9092d5dbcdcd4e2f4d4bb79fceaa2f9b8a;p=thirdparty%2Fgoogle%2Ffonts.git Big Shoulders Stencil 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-Stencil/sources/BigShouldersStencil.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/bigshouldersstenciltextsc/config.yaml b/ofl/bigshouldersstenciltextsc/config.yaml index f9d442b219..bdeb97872d 100644 --- a/ofl/bigshouldersstenciltextsc/config.yaml +++ b/ofl/bigshouldersstenciltextsc/config.yaml @@ -1,8 +1,8 @@ sources: - - ../Big-Shoulders-Stencil/sources/BigShouldersStencil.glyphs + - Big-Shoulders-Stencil/sources/BigShouldersStencil.glyphs recipe: - ../Big-Shoulders-Stencil/fonts/variable/text/BigShouldersStencilText[wght].ttf: - - source: ../Big-Shoulders-Stencil/sources/BigShouldersStencil.glyphs + Big-Shoulders-Stencil/fonts/variable/text/BigShouldersStencilText[wght].ttf: + - source: Big-Shoulders-Stencil/sources/BigShouldersStencil.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-Stencil/fonts/variable/text/BigShouldersStencilTextSC[wght].ttf: - - source: ../Big-Shoulders-Stencil/sources/BigShouldersStencil.glyphs + Big-Shoulders-Stencil/fonts/variable/text/BigShouldersStencilTextSC[wght].ttf: + - source: Big-Shoulders-Stencil/sources/BigShouldersStencil.glyphs - args: --filter ... --filter FlattenComponentsFilter --filter DecomposeTransformedComponentsFilter operation: buildVariable - operation: subspace diff --git a/ofl/bigshouldersstenciltextsc/upstream_info.md b/ofl/bigshouldersstenciltextsc/upstream_info.md index 6cf169c27e..089c0219e1 100644 --- a/ofl/bigshouldersstenciltextsc/upstream_info.md +++ b/ofl/bigshouldersstenciltextsc/upstream_info.md @@ -94,3 +94,12 @@ The upstream repository had a `Big-Shoulders-Stencil/sources/config.yml` at the The source block is complete and accurate. The repository URL and commit hash were verified against the onboarding PR #7789 and commit message. The override config.yaml is correctly present in the family directory, enabling the SC variant build from the upstream `.glyphs` source. The only minor note is that the `files` block in METADATA.pb references a binary path (`BigShouldersStencilTextSC[wght].ttf`) that did not exist at the referenced upstream commit. The font was actually built from source using the config.yaml recipe. This is a cosmetic issue and does not affect functionality. + + +## 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-Stencil/sources/BigShouldersStencil.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.