Skip to content
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

search in /usr/local/lib and /usr/lib like the system loader #167

Merged
merged 9 commits into from
Nov 16, 2022

Conversation

isuruf
Copy link
Collaborator

@isuruf isuruf commented Nov 9, 2022

Fixes #155 and fixes the issues in numpy/numpy#22525

@isuruf
Copy link
Collaborator Author

isuruf commented Nov 10, 2022

This is ready for review. mypy failure seems to be unrelated.

Copy link
Collaborator

@HexDecimal HexDecimal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. I trust the tests more than my ability to read the code right now. You can rebase/merge if you want to fix the Mypy error. I only have aesthetic suggestions for code changes.

I never knew startswith could take a tuple.

delocate/tests/test_libsana.py Outdated Show resolved Hide resolved
delocate/tests/test_libsana.py Outdated Show resolved Hide resolved
delocate/libsana.py Outdated Show resolved Hide resolved
@HexDecimal
Copy link
Collaborator

If this is blocking a project then I can make a new release shortly after this is merged.

@HexDecimal
Copy link
Collaborator

One more thing. This needs an addition to the changelog.

@codecov-commenter
Copy link

codecov-commenter commented Nov 10, 2022

Codecov Report

Merging #167 (55fb945) into master (60ab91b) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #167      +/-   ##
==========================================
+ Coverage   95.45%   95.47%   +0.02%     
==========================================
  Files          14       14              
  Lines        1077     1084       +7     
==========================================
+ Hits         1028     1035       +7     
  Misses         49       49              
Impacted Files Coverage Δ
delocate/libsana.py 97.74% <100.00%> (+0.09%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@isuruf isuruf merged commit 653b973 into matthew-brett:master Nov 16, 2022
@isuruf isuruf deleted the search_usr_local branch November 16, 2022 16:34
@isuruf isuruf restored the search_usr_local branch November 16, 2022 16:34
@isuruf isuruf deleted the search_usr_local branch December 15, 2022 22:02
@isuruf isuruf restored the search_usr_local branch December 17, 2022 19:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CMake-built dependency uses @rpath without setting it properly
3 participants