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
mono: re-enable parallel building #32386
Conversation
LGTM |
This appears to have broken |
Sigh quite the drag. |
Fixed in 90bcfc7 |
See #32386 -- while Mono in general should build correctly with parallel building, it seems the 4.4 branch has broken. Instead, allow parallel build support to be overridden by individual versions, and default to true. Signed-off-by: Austin Seipp <aseipp@pobox.com>
This probably needs some more work: |
Disabled for |
Disabled |
:-) sounds like it should be disabled in generic mono expression and explicitly enabled in versions tested to be OK. |
If practically all of them are broken, we should just back out the change? |
See NixOS#32386 -- while Mono in general should build correctly with parallel building, it seems the 4.4 branch has broken. Instead, allow parallel build support to be overridden by individual versions, and default to true. Signed-off-by: Austin Seipp <aseipp@pobox.com> (cherry picked from commit 90bcfc7)
See NixOS#32386 mono46: https://hydra.nixos.org/build/65617511 mono48: https://hydra.nixos.org/build/65600645 (cherry picked from commit 0a41fc6)
Motivation for this change
Shorter build times for mono packages. The commit that originally disabled parallel builds is from 2012 for an older version of mono and according to upstream the issues have been since resolved - https://bugzilla.xamarin.com/show_bug.cgi?id=2018.
/cc maintainers @viric @thoughtpolice @obadz @vrthra
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)