admin/brew
1
0
Fork 0
Code Issues Packages Projects Releases 6 Wiki Activity

Merge pull request #15722 from carlocab/julia-rpath-fix

Browse Source 
os/mac/keg_relocate: avoid rpath re-ordering
This commit is contained in:
Carlo Cabrera 2023-07-20 23:21:04 +08:00 committed by GitHub
commit 8f5609f678
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
Library/Homebrew/extend/os/mac/keg_relocate.rb
View File

@ -57,7 +57,13 @@ class Keg
each_linkage_for(file, :rpaths) do |bad_name|
# Strip duplicate rpaths and rpaths rooted in the build directory.
if rooted_in_build_directory?(bad_name) || (file.rpaths.count(bad_name) > 1)
if rooted_in_build_directory?(bad_name) ||
(file.rpaths(resolve_variable_references: false).count(bad_name) > 1)
# TODO: Drop the `resolve_variable_references` argument above (defaults to `true`)
# and fix `delete_rpath` so that it deletes the *last* LC_RPATH command
# (instead of the first one) to avoid changing LC_RPATH command ordering.
# NOTE: `delete_rpath` will also need to be able to handle rpaths that are different
# strings but resolve to the same location.
delete_rpath(bad_name, file)
else
new_name = opt_name_for(bad_name)