-
Notifications
You must be signed in to change notification settings - Fork 284
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
avoid CMake fiddling with the RPATHs injected by EasyBuild via --rpath #1031
Conversation
@wpoely86 please review? |
self.lib_exts = [] | ||
|
||
def configure_step(self, *args, **kwargs): | ||
"""Configure build using 'make config' (only for recent versions (>= v5)).""" | ||
|
||
if LooseVersion(self.version) >= LooseVersion("5"): | ||
|
||
if build_option('rpath'): | ||
# patch Makefile to tell CMake not to wipe the RPATHs we inject... | ||
apply_regex_substitutions('Makefile', [(r'^(CONFIG_FLAGS\s*=\s*)', r'\1-DCMAKE_SKIP_RPATH=ON ')]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no space between \1
and -D...
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be there already, but doesn't hurt to make sure, you're right, I'll add it.
Thanks for the review @wpoely86! |
Passing
-DCMAKE_SKIP_RPATH=ON
when using--rpath
is required to avoid that CMake strips all RPATHs from the built binaries/libraries before actually installing them (which it does by default, urgh).Required to get METIS to build in conjunction with
--rpath
(cfr. easybuilders/easybuild-framework#1942).