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
BUG: fix all RPATHs that start with $ORIGIN #510
Conversation
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.
Thanks. This looks good. Just one comment.
Well, two actually. The commit message should be something like: BUG: fix all RPATHs that start with $ORIGIN
or something similar, where the BUG:
prefix is the important change. It would also be great to explain the motivation for the change in the commit message (something similar to the description of the issue). And it should contain Fixes: #509.
as last line.
This fixes only the the RPATH handling for ELF binaries on Linux. The macOS code needs a similar change. |
bf2f0f7
to
98d3544
Compare
When shared libraries are build in a project subdir or in a subproject, the RAPTH added by meson pointing to the shared library build directory is not just '$ORIGIN/' but a longer relative path. Apply the same fix to all RPATHs that start with '$ORING/'. Fixes mesonbuild#509.
I've pushed a patch that fixes the identified issues. Please take a look. |
Thanks for the fix! LGTM |
meson-python 0.14.0 has a bug that handles rpath of python binding incorrectly. Use 0.13.2 temporarily as a fix. The upstream fix has landed in mesonbuild/meson-python#510
Attempt to fix #509