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
cc-wrapper: Use stdenvNoCC to build #29617
Conversation
b6195d0
to
7e04d57
Compare
7e04d57
to
1549b37
Compare
cc-wrapper may wrap a cc-compiler, but it doesn't need one to build itself. (c.f. expand-response-params is a separate derivation.) This helps avoid cycles on the cross stuff, in addition to removing a useless dependency edge. I could have been super careful with overrides in the stdenv to avoid the mass rebuild, but I don't think it's worth it.
1549b37
to
d349f9a
Compare
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.
Looks OK to me, both in the idea and implementation.
@@ -1,15 +1,13 @@ | |||
{ lib | |||
, crossSystem, config | |||
, crossSystem, config, overlays |
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.
Hmm, this was missing since 92edcb7 and noone has encountered/reported the error, I guess, as we currently only use this file for "x86_64-solaris".
Motivation for this change
cc-wrapper may wrap a C compiler, but it doesn't need one to build itself. (c.f. expand-response-params is a separate derivation.) This helps avoid cycles on the cross stuff, in addition to removing a useless dependency edge.
I could have been super careful with overrides in the stdenv to avoid the mass rebuild, but I don't think it's worth it. [If all goes well, we'll merge this with the other PRs in https://github.com/NixOS/nixpkgs/projects/8, so they'll be a mas-rebuild anyways.]
Things done
Building stdenv as part of other things. PR has more commits than it should because it depends on other PRs.
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)