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
valgrind: Disable parallel building #51082
Conversation
Success on x86_64-linux (full log) Attempted: valgrind Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: valgrind Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: valgrind Partial log (click to expand)
|
LGTM, I've tracked down the issue earlier today to a bug either in automake or how valgrind thinks automake works, and disabling parallel building should improve things in the meantime (from my limited understanding of the problem). |
Oh hey, actually, it's fixed upstream: https://sourceware.org/git/?p=valgrind.git;a=commit;h=7820fc268fae4353118b6355f1d4b9e1b7eeebec Should we instead cherry-pick this and keep parallel building? |
Motivation for this change
Valgrind appears to have a race condition with an unspecified dependency somewhere in the Makefile. I'll see about contacting upstream about this as well, but in the meantime, we can unblock ourselves.
#50876 (comment)
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)