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

Update ROOT_LIBRARY_PATH in thisdd4hep.sh #1408

Merged
merged 3 commits into from
Mar 5, 2025

Conversation

veprbl
Copy link
Contributor

@veprbl veprbl commented Mar 3, 2025

BEGINRELEASENOTES

Copy link

github-actions bot commented Mar 3, 2025

Test Results

   16 files     16 suites   7h 5m 23s ⏱️
  370 tests   370 ✅ 0 💤 0 ❌
2 887 runs  2 887 ✅ 0 💤 0 ❌

Results for commit 2b8cbcd.

♻️ This comment has been updated with latest results.

@andresailer
Copy link
Member

When is this needed for cling? This would not affect the creation of the rootmaps during the compilation of DD4hep.

@ybedfer
Copy link
Contributor

ybedfer commented Mar 4, 2025

I can add that this commit fixes my issue #1407.

@andresailer
Copy link
Member

If we move the setting of ROOT_LIBRARY_PATH to

dd4hep_add_library_path() {
local p=${1};
if [ @APPLE@ ]; then
# Do not prepend system library locations on macOS. Stuff will break.
[[ "$p" = "/usr/lib" || "$p" = "/usr/local/lib" ]] && return
dd4hep_add_path DYLD_LIBRARY_PATH "$p"
dd4hep_add_path DD4HEP_LIBRARY_PATH "$p"
else
dd4hep_add_path LD_LIBRARY_PATH "$p"
fi
}

also the packages depending on DD4hep benefit from this change directly

@veprbl
Copy link
Contributor Author

veprbl commented Mar 4, 2025

When is this needed for cling? This would not affect the creation of the rootmaps during the compilation of DD4hep.

cppyy is using cling

@andresailer andresailer merged commit 8545739 into AIDASoft:master Mar 5, 2025
16 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.

3 participants