-
Notifications
You must be signed in to change notification settings - Fork 79
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
Adding rpath support #131
Adding rpath support #131
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 seems like a good idea, but instead of supporting specific linker arguments it would probably make more sense to pass through all -Wl
arguments as-is to the linker. In addition to -rpath
there's for example also -rpath-link
which is quite useful to have.
Can you update the PR?
Seems like a decent idea. |
Hopefully the I'll update and worst case we strip it out. |
I'm not quite sure if this project wants to support it, but it would be a nice to have in my opinion.