Skip to content

Fix MacOS rpath handling#61

Merged
abgandar merged 1 commit intomasterfrom
isinf
Mar 22, 2025
Merged

Fix MacOS rpath handling#61
abgandar merged 1 commit intomasterfrom
isinf

Conversation

@abgandar
Copy link
Collaborator

To allow both the traditional installation of DACE libraries in system directories and the full in-tree build using FetchContent on MacOSX the logic for RPATH handling is now this:

  • build tree: all libraries and binaries are built with full absolute RPATH pointing to the binary tree. This allows any executable linking the dace target to "just run" in place. They are, however, not relocatable
  • install tree: all absolute paths are replaced with @rpath to allow relocation and installation into system directories. This allows using find_package() to link DACE programs that are location independent

@abgandar
Copy link
Collaborator Author

In future major releases, we should consider removing the dynamic library option entirely due to the messiness and limited utility (library locations across platforms, C++ library inconsistencies, C++ templates, small size of full library)

@abgandar abgandar merged commit 38d46a0 into master Mar 22, 2025
5 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