-
-
Notifications
You must be signed in to change notification settings - Fork 15.5k
vagrant: 2.1.2 -> 2.2.0 + vagrant-libvirt: init at 2.2.0 #49220
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
Conversation
Adds a flag and a new package to build vagrant with libvirt support, to be able to install and use the vagrant-libvirt plugin.
@aneeshusa, @globin please review. |
Perhaps it is not needed/wanted to add the withLibvirt flag and always prepare vagrant to be able to run with libvirt, idk, please advice. |
@GrahamcOfBorg build libvirt vagrant-libvirt |
Like discussed with @globin and @lheckemann i changed the derivation to default to |
awesome work @calbrecht! |
@GrahamcOfBorg build vagrant |
This comment has been minimized.
This comment has been minimized.
Success on x86_64-darwin (full log) Attempted: vagrant Partial log (click to expand)
|
This comment has been minimized.
This comment has been minimized.
Success on x86_64-darwin (full log) Attempted: vagrant Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: vagrant Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: vagrant Partial log (click to expand)
|
This comment has been minimized.
This comment has been minimized.
Success on aarch64-linux (full log) Attempted: vagrant Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: vagrant Partial log (click to expand)
|
This does double the closure size of vagrant. Not sure if that's a problem.
|
I don't think that should be too critical for vagrant since it will never be a light-weight application and it is possible to disable it easily. |
How does this work? I tried running
|
@mat8913 with the PR, libvirt plugin is already installed with vagrant itself. But please make sure that you build from master branch of nixpkgs because the PR was not backported to stable. |
@mat8913 Epic. Thanks for your work then. :) |
Motivation for this change
vagrant plugin install vagrant-libvirt
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip --against 591ba7335bae245733777c1c4eeedb4d34ef57af"
./result/bin/
)nix path-info -S
before and after)