Skip to content

[build] Allow searching for external FreeType on MacOS#21389

Open
vepadulano wants to merge 1 commit intoroot-project:masterfrom
vepadulano:apple-do-not-force-builtin-freetype
Open

[build] Allow searching for external FreeType on MacOS#21389
vepadulano wants to merge 1 commit intoroot-project:masterfrom
vepadulano:apple-do-not-force-builtin-freetype

Conversation

@vepadulano
Copy link
Member

And fallback to using the builtin if the external can't be found.

Useful when there is an externally-provided FreeType library on MacOS, such as on conda-forge.

This is an upstream of https://github.com/conda-forge/root-feedstock/blob/df86316b299d8214d3181d76c31925696f076d5e/recipe/patches/0014-Force-SDKROOT.patch

@vepadulano vepadulano requested a review from dpiparo February 26, 2026 09:30
@vepadulano vepadulano self-assigned this Feb 26, 2026
@vepadulano vepadulano requested a review from bellenot as a code owner February 26, 2026 09:30
@vepadulano vepadulano added in:Build System clean build Ask CI to do non-incremental build on PR labels Feb 26, 2026
And fallback to using the builtin if the external can't be found.

Useful when there is an externally-provided FreeType library on MacOS, such as on conda-forge.

This is an upstream of https://github.com/conda-forge/root-feedstock/blob/df86316b299d8214d3181d76c31925696f076d5e/recipe/patches/0009-disable-builtin-freetype-macos.patch
@vepadulano vepadulano force-pushed the apple-do-not-force-builtin-freetype branch from 018844b to 0faedfd Compare February 26, 2026 09:31
@github-actions
Copy link

Test Results

    22 files      22 suites   3d 7h 22m 36s ⏱️
 3 801 tests  3 798 ✅ 1 💤 2 ❌
76 499 runs  76 488 ✅ 9 💤 2 ❌

For more details on these failures, see this check.

Results for commit 0faedfd.

@guitargeek
Copy link
Contributor

Great! This change actually shows that the fail-on-missing mechanism was not implemented for the builtin_freetype. Can you maybe add it in this PR, copying some code for other builtins? The idea is that no builtin is implicitly enabled if you set fail-on-missin=ON. The configuration should fail then with an error that tells you which builtin you have to enable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clean build Ask CI to do non-incremental build on PR in:Build System

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants