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
llvm_{38,39}: unconditionally set triples, not just w/musl #42463
llvm_{38,39}: unconditionally set triples, not just w/musl #42463
Conversation
Follow-on to same changes made to other versions: See NixOS#42454, NixOS#42456.
Maybe start here since (I think?) these are lowest impact? I think Darwin builder might be able to give us feedback now... let's see! :) |
@GrahamcOfBorg build tests.cc-wrapper-libcxx-39 tests.cc-wrapper-clang-39 |
Success on aarch64-linux (full log) Attempted: tests.cc-wrapper-libcxx-39, tests.cc-wrapper-clang-39 Partial log (click to expand)
|
Failure on x86_64-darwin (full log) Attempted: tests.cc-wrapper-libcxx-39, tests.cc-wrapper-clang-39 Partial log (click to expand)
|
Well didn't have fork failure, but timed out d'oh haha. |
Success on x86_64-linux (full log) Attempted: tests.cc-wrapper-libcxx-39, tests.cc-wrapper-clang-39 Partial log (click to expand)
|
This (and same changes to other versions) was requested to be made more general than musl, |
I should have "moved" the flags instead of "copying" them earlier :).
Follow-on to same changes made to other versions:
See #42454, #42456.
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)