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
llvm4: unconditionally set triples, not just on musl. #42456
Conversation
Split off from NixOS#42454 as requested.
@GrahamcOfBorg build tests.cc-wrapper-libcxx-4 tests.cc-wrapper-clang-4 |
Failure on x86_64-darwin (full log) Attempted: tests.cc-wrapper-libcxx-4, tests.cc-wrapper-clang-4 Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: tests.cc-wrapper-libcxx-4, tests.cc-wrapper-clang-4 Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: tests.cc-wrapper-libcxx-4, tests.cc-wrapper-clang-4 Partial log (click to expand)
|
Follow-on to same changes made to other versions: See NixOS#42454, NixOS#42456.
@GrahamcOfBorg build tests.cc-wrapper-libcxx-4 tests.cc-wrapper-clang-4 |
Success on x86_64-linux (full log) Attempted: tests.cc-wrapper-libcxx-4, tests.cc-wrapper-clang-4 Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: tests.cc-wrapper-libcxx-4, tests.cc-wrapper-clang-4 Partial log (click to expand)
|
Failure on x86_64-darwin (full log) Attempted: tests.cc-wrapper-libcxx-4, tests.cc-wrapper-clang-4 Partial log (click to expand)
|
Well seems to have worked for llvm 3.8, 3.9--or at least no obvious fires were started as a result :). Darwin looks good: https://hydra.nixos.org/build/76526688 (might not be first build containing change, but look at log to see we're setting triples as expected and such). Let's try 4! |
Split off from #42454 as requested.
Boxes I've checked below aren't actually for this exact commit,
but the original I sent to master and actually the one I tested on
a slightly older master before moving to staging.
Basically these really should apply but who can say :).
Build in-flight anyway :).
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)