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
ld-wrapper: Use .dylib not .so on Darwin #29317
Conversation
ld-wrapper in a few places assumes the extension of a dynamic lib. This makes it accurate on Darwin.
Why wouldn't you change the Sierra hack to use this in a second commit in this pull request? |
@orivej I need to forward port it from 17.03 first. |
We're getting a disallowed |
Not sure if it's important, but there are a bunch of libraries that don't follow the convention for darwin and hardcode |
Are there any updates on this pull request, please? |
Thank you for your contributions.
|
Is this still relevant on Big Sur? |
Well there seem to be both .so and .dylib on darwin depending on the package bothering to be conditional. Someone from @NixOS/darwin-maintainers should say whether whatever functionality ld-wrapper was doing here is useful for non-ELF, and if so I'll revive it else I'll just close it. |
I marked this as stale due to inactivity. → More info |
Closing this as it is abandoned. |
Motivation for this change
ld-wrapper in a few places assumes the extension of a dynamic lib. This makes it accurate on Darwin.
My sierra hack will use this. That I'd like to back-port to 17.09, so this should be too.
Things done
We could be relying on ld-wrapper not working correctly as intended in myriad ways. Best to test this one pretty well.
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)CC @orivej