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
nixos/virtualbox-image: add params #80736
Conversation
@GrahamcOfBorg test virtualbox |
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.
Wow, thank you so much <3
example = { | ||
audio = "alsa"; | ||
rtcuseutc = "on"; | ||
usb = "off"; |
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.
Might make sense to be able to specify usb = false;
and have that converted to --usb=off
on its own, which is a small detour from lib.cli.toGNUCommandLine
behavior.
(This is obviously a reverse compatible change and can be addressed later if desired).
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.
Ah, good point. Let me find out how that is possible.
I'm inclined to merge this as-is and will do so later today unless anyone feels otherwise :) |
CC @aszlig as someone who might be interested in this change. |
I've implemented |
Oh, I had a prototype in works. Something like this, for inspiration.
|
Motivation for this change
The following change allows users to redefine the default parameters that are being passed to the VirtualBox appliance.
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)