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
virtualbox: 5.2.2 -> 5.2.4 #32983
virtualbox: 5.2.2 -> 5.2.4 #32983
Conversation
This package is especially sensitive to |
@orivej Of course you're right, missed them 😊 Tests also showed I forgot to update the The tests seemed to I have no idea why @GrahamcOfBorg complains about xen here, maybe @grahamc has an idea? |
Thank you! The tests were broken by 0d27df2 (as can be seen at https://hydra.nixos.org/build/65928567). /cc @dezgeg (not to blame, but to inform). |
0e869ac
to
65f1325
Compare
With devtmpfs introduced in 0d27df2 it is created automatically. See NixOS#32983 (comment)
65f1325
to
0543dc7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have fixed this up a bit and consider it ready to merge.
@GrahamcOfBorg eval (sorry for the noise, master was broken by a merge) |
Motivation for this change
This bumps virtualbox to version 5.2.4, adds its license and me as a maintainer.
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)