GNU: use -Wl,-rpath,<dir> instead of -Wl,-R<dir> #4060
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes #4060
Summary of changes
Use
-rpath
instead of-R
when using binutils.The latter is supported in binutils for backwards compatibility, but in
general
-R<path>
is equivalent to--just-symbols=<path>
whenpath
is a file; only when it's a directory, it's treated as
-rpath=<path>
.Better avoid that ambiguity and use
-rpath
.Also split
-Wl,--enable-new-dtags
and-Wl,-rpath,...
into twoseparate arguments, which is more common, and more likely to be parsed
correctly by compiler wrappers.
This commit does not attempt to add
--enable-new-dtags
to otherlinkers than binutils ld/gold that support the flag.
Pull Request Checklist
newsfragments/
].