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.14 -> 5.2.22 #51669
virtualbox: 5.2.14 -> 5.2.22 #51669
Conversation
I had to drop xorriso because it didn't seem to want to compile with it any more, and had to add libopus as a build input because it wouldn't compile without that.
Looks like the URL format has changed. Or maybe this is just the first version and doesn't have a revision? I don't know.
@GrahamcOfBorg test virtualbox |
@GrahamcOfBorg build nixosTests.virtualbox |
|
I think this broke VirtualBox-GuestAdditions unfortunately. The last upgrade fixed up the patches apparently: dd53d0f. cc @NeQuissimus authored that last upgrade. |
Eep. Maybe should just revert then until somebody who is much smarter than me can fix the patches? |
Maybe. The failure is a bit weird, I'm not sure why it the whole patchPhase isn't logged. But I'm guessing this must (edit: it isn't) be the failing line:
So it could be that we just need to change the directory name? |
Scratch that, failure does seem to happen in the patchPhase 😕 (can't get any output after). |
Ah, just need to update the src hash. It used the source from 5.2.20 since it was cached. |
That fixes the problem:
|
I had to drop xorriso because it didn't seem to want to compile with it any more, and had to add libopus as a build input because it wouldn't compile without that.
Motivation for this change
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)