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
[staging] rust: cleanup, use correct llvm version #106886
Conversation
Fails to build on darwin.
|
a120f3e
to
091400e
Compare
091400e
to
c73d96f
Compare
@zowoq Thx for testing, didn't have any darwin machine to test on. I've reverted those changes but in a way that is more obvious what is used for, hope it is ok. |
Builds thunderbird on Linux. Can someone check building rustc on macOS? cc @dasJ |
Result of 220 packages marked as broken and skipped:
518 packages failed to build:
|
|
@dasJ This doesn't look related to rustc changes. Can you maybe cherry-pick it to latest master and try there? Thanks! |
Built |
There was an attempt to remove this in NixOS#106886 [1], but there was a darwin compilation failure. Whatever caused the failure then is no longer a problem now. [1] NixOS#106886
An attempt to remove it was made in NixOS#106886, but there was a build failure. Since that failure no longer occurs, remove it now. Based on thefloweringash/nixpkgs@1904236.
Motivation for this change
I've noticed rustc 1.48 uses LLVM 10 while version 11 is set in all-packages:
This change properly propagates correct runtime LLVM version.
I've also removed bootstrap overridefor clang based stdenv, which should be unnecessary, I didn't test it.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)