-
Notifications
You must be signed in to change notification settings - Fork 69
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
Add runtime-path to DynamicLinking.md #246
Conversation
@sbc100 are there other people I should ask to review this? |
I wonder of @dicej has any thoughts here, from the perspective of componentize-py. |
This is important for truly-at-runtime dynamic linking, but I'd think not really necessary for "componentize" dynamic linking. Maybe if there is an "install" step before componentizing it could be easier to use the rpath, but nothing that couldn't be managed by patching around the build scripts a bit. But I agree it'd be interesting to hear @dicej's opinion. |
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.
lgtm % comments from others.
Adding @dschuff too
Yeah, I'm not sure how |
Changes to the conventions for #244.
cc @ryanking13 @sbc100.