Skip to content

[cmake] Require explicit opt-in to use the builtin openssl#21689

Merged
dpiparo merged 1 commit intoroot-project:masterfrom
dpiparo:no_fallback_for_openssl
Mar 26, 2026
Merged

[cmake] Require explicit opt-in to use the builtin openssl#21689
dpiparo merged 1 commit intoroot-project:masterfrom
dpiparo:no_fallback_for_openssl

Conversation

@dpiparo
Copy link
Member

@dpiparo dpiparo commented Mar 25, 2026

As the title of the PR says, a user can decide to build ROOT against the builtin openssl, however that must be a conscious decision.

@dpiparo dpiparo requested a review from hageboeck March 25, 2026 11:27
@dpiparo dpiparo self-assigned this Mar 25, 2026
@dpiparo dpiparo marked this pull request as ready for review March 25, 2026 11:27
@dpiparo dpiparo requested a review from bellenot as a code owner March 25, 2026 11:27
@dpiparo dpiparo marked this pull request as draft March 25, 2026 14:19
@dpiparo dpiparo force-pushed the no_fallback_for_openssl branch from 4117477 to 909c760 Compare March 25, 2026 14:24
@dpiparo dpiparo marked this pull request as ready for review March 25, 2026 15:25
@dpiparo dpiparo requested a review from guitargeek March 25, 2026 15:26
@github-actions
Copy link

github-actions bot commented Mar 25, 2026

Test Results

    22 files      22 suites   3d 2h 54m 23s ⏱️
 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 909c760.

♻️ This comment has been updated with latest results.

@dpiparo dpiparo merged commit e70dedd into root-project:master Mar 26, 2026
50 of 53 checks passed
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.

1 participant