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
treewide: Clean up multi-arch tools and usage #33087
Conversation
Clang stdenvs with either libc++ should work now.
aa7690f
to
b6afa3f
Compare
@nixborg build |
Jobset created at https://hydra.mayflower.de/jobset/nixos/pr-33087 |
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 good
pkgs/top-level/all-packages.nix
Outdated
# Include unwrapped binaries like AS, etc. and remove libcxx/libcxxabi | ||
extraBuildInputs = [ stdenv.cc.cc ]; | ||
# Remove libcxx/libcxxabi | ||
extraBuildInputs = [ ]; |
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.
This breaks tests.cc-wrapper-gcc
on darwin, but there's probably a nicer way to fix it.
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 did the same method, but made it darwin-specific and specifically clang.
- Dispatch off more appropriate conditions---`stdenv.cc.is*` and `hostPlatform.is*` directly---rather than the OS as a proxy. - Don't worry about pulling in binutils from normal `stdenv.cc` for `gccMultiStdenv`. - Defining a `multiStdenv` that uses whatever compiler is default. - Define `stdenv_32bit` in terms of `multiStdenv`.
- Don't needlessly force GCC - Prefer using multi-arch stdenvs to using multi-arch tools directly.
b6afa3f
to
558c96f
Compare
Motivation for this change
See the commit messages for more details, but the principles basically are:
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)