Skip to content

[CMake] Append to CMAKE_INSTALL_RPATH with CACHE FORCE#19135

Merged
guitargeek merged 1 commit into
root-project:masterfrom
guitargeek:issue-19134
Jun 24, 2025
Merged

[CMake] Append to CMAKE_INSTALL_RPATH with CACHE FORCE#19135
guitargeek merged 1 commit into
root-project:masterfrom
guitargeek:issue-19134

Conversation

@guitargeek
Copy link
Copy Markdown
Contributor

Correctly append the additional RPATHs set by the ROOT build system to the CMAKE_INSTALL_RPATH variable set by the user, which is cached in CMAKE_INSTALL_RPATH-CACHED by the ROOT build system.

Follows up on c77bd87.

Closes #19134.

Needs a backport to 6.36.

Correctly append the additional RPATHs set by the ROOT build system to
the `CMAKE_INSTALL_RPATH` variable set by the user, which is cached in
`CMAKE_INSTALL_RPATH-CACHED` by the ROOT build system.

Follows up on c77bd87.

Closes root-project#19134.

Needs a backport to 6.36.
@github-actions
Copy link
Copy Markdown

Test Results

    20 files      20 suites   3d 22h 9m 14s ⏱️
 3 042 tests  3 042 ✅ 0 💤 0 ❌
59 059 runs  59 059 ✅ 0 💤 0 ❌

Results for commit 90f5a4c.

@dpiparo
Copy link
Copy Markdown
Member

dpiparo commented Jun 23, 2025

Thanks for these changes!

Comment thread cmake/modules/RootBuildOptions.cmake
ferdymercury added a commit to ferdymercury/root that referenced this pull request Jun 24, 2025
@guitargeek guitargeek merged commit 30aff70 into root-project:master Jun 24, 2025
25 checks passed
@guitargeek guitargeek deleted the issue-19134 branch June 24, 2025 22:23
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.

Installation issue for ROOT 6.36 RPATHs through Spack on MacOS

4 participants