Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CMake] Make it possible to pass existing CLAD_SOURCE_DIR #16584

Merged
merged 1 commit into from
Oct 2, 2024

Conversation

guitargeek
Copy link
Contributor

This would address two annoyances I have at the same time:

  1. One can now clone both root and clad, linking the source directories together. This makes it much easier to try out changes in clad in the context of ROOT.

  2. ROOT can now configure and build without an internet connection, even with Dclad=ON. This partially addresses Disable automatic 'call home' in cmake when not needed #11603.

This would address two annoyances I have at the same time:

  1. One can now clone both `root` and `clad`, linking the source
     directories together. This makes it much easier to try out changes
     in clad in the context of ROOT.

  2. ROOT can now configure and build without an internet connection,
     even with `Dclad=ON`. This partially addresses root-project#11603.
@guitargeek guitargeek self-assigned this Oct 2, 2024
@guitargeek guitargeek requested a review from bellenot as a code owner October 2, 2024 16:10
Copy link

github-actions bot commented Oct 2, 2024

Test Results

     9 files       9 suites   1d 10h 10m 0s ⏱️
 2 701 tests  2 701 ✅ 0 💤 0 ❌
22 779 runs  22 779 ✅ 0 💤 0 ❌

Results for commit b08905a.

@guitargeek
Copy link
Contributor Author

Thanks for the review!

@guitargeek guitargeek merged commit 2ec063a into root-project:master Oct 2, 2024
19 of 20 checks passed
@guitargeek guitargeek deleted the clad_src_dir branch October 2, 2024 22:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants