Kyle Benesch

Results 315 comments of Kyle Benesch

I was just hoping that [something here in the docs](https://macholib.readthedocs.io/en/latest/dyld.html) could replace a lot of the library search code I wrote, but `@rpath` isn't mentioned there.

For clarity: `arm64` links to 3 more libraries than `x86_64`. I'm not experienced enough the explain why. ``` { 'x86_64': [ ('/System/Library/Frameworks/Carbon.framework/Versions/A/Carbon', '2.0.0', '165.0.0'), ('/System/Library/Frameworks/Security.framework/Versions/A/Security', '1.0.0', '60158.100.133'), ('/System/Library/Frameworks/Foundation.framework/Versions/C/Foundation', '300.0.0', '1858.112.0'),...

A lot has changed. Can you test your wheel on the latest version of delocate?

Looks like exclusion is performed too late in the process of finding a library. It's done after rpaths are resolved which causes the error seen here. I think `copy_filt_func` should...

> * A "bare" call of delocate complains that these shared objects use two libgfortran shared objects: the one from the gfortran compiler and the one from openblas. > >...

Honestly I think the whole source should be converted to use `pathlib`'s `Path` and `PurePosixPath`.

This kind of stuff I want to revisit by adding Ruff's [PTH](https://docs.astral.sh/ruff/rules/#flake8-use-pathlib-pth) rule. I don't want these outdated calls to be refactored into *other* outdated calls.

> I was interested to see that there are CI tests running on windows. I don't know the full history of this project but I guess there was some cross-platform...

> I can try to reproduce the issue again tomorrow. Unfortunately I don't actually have a Windows machine so it's a lot of pushing and waiting for CI. :( True...

I apologize, but I'd like to put any Windows specific refactoring on hold for now. At least until certain Ruff rules are enforced and older versions of Python are dropped.