Skip to content

[cmake] Builtins: more uniform CMake configurations#21688

Merged
dpiparo merged 4 commits intoroot-project:masterfrom
dpiparo:unifirm_builtin_bundle
Mar 26, 2026
Merged

[cmake] Builtins: more uniform CMake configurations#21688
dpiparo merged 4 commits intoroot-project:masterfrom
dpiparo:unifirm_builtin_bundle

Conversation

@dpiparo
Copy link
Copy Markdown
Member

@dpiparo dpiparo commented Mar 25, 2026

No description provided.

@dpiparo dpiparo requested a review from hageboeck March 25, 2026 10:51
@dpiparo dpiparo self-assigned this Mar 25, 2026
@ferdymercury
Copy link
Copy Markdown
Collaborator

Thanks!!
Wrt last commit, there is this a-bit-related discussion on how much automation the builtin mechanism should have (#18413)
and the still problematic open JIRA Issue https://its.cern.ch/jira/browse/ROOT-10743

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 25, 2026

Test Results

    22 files      22 suites   3d 4h 58m 15s ⏱️
 3 835 tests  3 833 ✅  1 💤 1 ❌
75 693 runs  75 674 ✅ 18 💤 1 ❌

For more details on these failures, see this check.

Results for commit 3787a90.

♻️ This comment has been updated with latest results.

@dpiparo dpiparo force-pushed the unifirm_builtin_bundle branch from a658760 to b1db9e9 Compare March 25, 2026 11:14
in case the library is not found on the system.
@dpiparo dpiparo force-pushed the unifirm_builtin_bundle branch from b1db9e9 to 3787a90 Compare March 25, 2026 15:08
@dpiparo dpiparo marked this pull request as ready for review March 25, 2026 18:25
@dpiparo dpiparo requested a review from bellenot as a code owner March 25, 2026 18:25
@dpiparo
Copy link
Copy Markdown
Member Author

dpiparo commented Mar 25, 2026

The failure on ubu24 is unrelated

@dpiparo dpiparo merged commit 325181e into root-project:master Mar 26, 2026
28 of 30 checks passed
@hahnjo
Copy link
Copy Markdown
Member

hahnjo commented Mar 30, 2026

I think this logic is flawed: It makes mandatory FreeType, GIF, JPEG, PNG which are not needed for a minimal build...

@dpiparo
Copy link
Copy Markdown
Member Author

dpiparo commented Mar 30, 2026

and you are right. This was not intended.

dpiparo pushed a commit that referenced this pull request Mar 30, 2026
harz05 pushed a commit to harz05/root that referenced this pull request Apr 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants