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
rPackages: fix binutils path on Darwin #40360
Conversation
@Ericson2314 is this intensional? If so what's the difference between |
binutils is GNU binutils, darwin.cctools is Apple's linker, darwin.binutils is a mashup of the two so linking is correct but other GNU binutils are also present. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd just make them all darwin.binutils
, because that was how it evaluated before. But the bottom two substitutions are fine.
b5e978e
to
d35604d
Compare
@Ericson2314 Thank you for clarification! |
Are the subs necessary in the first place? install_name_tool is aleeady in stdenv. |
They are required. See #34766 |
Ok: we are probably missing This works for me at least:
|
Right. Adding |
Oh sorry! I didn't realize this was needed in the user environment. Either this or wrapping the runtime is probably the best route. |
Motivation for this change
Recent nixpkgs seems to have changed the package in which
otool
andinstall_name_tool
are living.Things done
Changed the relevant package paths.
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)