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
stdenv: better names for cc and bintools #39458
Conversation
Success on x86_64-linux (full log) Attempted: gcc Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: gcc Partial log (click to expand)
|
Thanks! It was quite ugly before. Just one thing: Multi-target compilers like clang need single-target wrappers, so the wrappers need to atleast add the target if one isn't already there. (Make sure not to make Probably also good to change GCC so it also has a targetPrefix attribute for consistency. |
f0978cc
to
5603d96
Compare
Fixed.
Too lazy :) |
... binutils and gcc add it already anyway. Without this it's easy to get cross-toolchain paths longer than 256 chars and nix-daemon will then fail to commit them to /nix/store on XFS.
5603d96
to
43dbc26
Compare
Success on aarch64-linux (full log) Attempted: gcc Partial log (click to expand)
|
Acceptable :) |
Success on x86_64-linux (full log) Attempted: gcc Partial log (click to expand)
|
Motivation for this change
Mostly beautification, but with a long
crossSystem.config
I managed to breaknix-build
on XFS without this. See commit messages.Things done