-
Notifications
You must be signed in to change notification settings - Fork 177
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
[nvq++] Add RPATH flags only to the final binary #52
Conversation
How do we get rid of all these spelling check warnings? |
Per the spell check warnings, I just merged the PR on that one. |
4fe6470
to
bd8ec11
Compare
In the three dots, next to the view check for files, you can choose the disable to option that |
bd8ec11
to
a277929
Compare
@schweitzpgi In general, add words that are spelled correctly but not recognized (such as e.g. "qubit") to the allow list |
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. Thank you for figuring this one out, Bruno.
a277929
to
68b618e
Compare
Description
This changes the behavior of the driver to only add the
-rpath
flags when linking the final binary. The main reason for this change is that the macOS linker does not allow adding-rpath
option when merging object files. But also, when building in Linux, it seems unnecessary to add a-rpath
flags when merging objects files (I think they get ignored anyway), and then again when doing the final binary liking.