Ignore duplicate rpaths in #change_rpath#438
Conversation
|
Even the existing test that I've just deleted seems to differ from |
This causes a deviation in behaviour from `install_name_tool`, which will happily change an existing rpath to one that already exists. Closes Homebrew#436.
42714f6 to
3275fec
Compare
|
Hmm, I wonder if |
|
I was wondering that too! But I don't have access to an old Mac to check. |
|
Neither do I anymore, unfortunately. I remember we did quite a bit of behavioral testing against |
|
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
|
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
|
Have you had the chance to think about how you want to handle this, @woodruffw? |
|
Sorry for the delay! And yeah: I think I agree that this change is for the best. |
|
No apologies necessary, and thanks! |
This causes a deviation in behaviour from
install_name_tool, whichwill happily change an existing rpath to one that already exists.
Closes #436.