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: 6.1.6 -> 6.1.10 #89798
virtualbox: 6.1.6 -> 6.1.10 #89798
Conversation
supersedes pull request #88172 |
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.
Looks good.
Build succeeds including kernel 5.7 and VMs are running fine.
.. is there someone that needs to be pinged, or something else that needs to be done for this one to be merged?? |
@gloaming i've been running my work VM for a while now using this derivation, and as far as I've been able to tell, there's no regression (auto scaling is as crappy as in any version beyond 6.1.2, though, but there's no obvious regression compared to previous version at least). |
@blitz just wanted to check back on https://github.com/NixOS/nixpkgs/pull/85772#issuecomment-617769606… were you able to take a look at the tests? |
6.1.12 was released yesterday. |
@flokli No luck with the tests yet. I'll take some time tomorrow to look into that again. That would simplify testing a lot... |
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've manually tested:
- Windows 10
- NixOS Plasma LiveCD
Looks great!
Alright, merged this in - let's leave 6.1.12 and the tests to (a) separate PR(s). |
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)