[Buildroot] [PATCH 1/1] Add libvirt (KVM) support

Thomas Petazzoni thomas.petazzoni at bootlin.com
Sat Apr 13 20:21:37 UTC 2019


Hello Christopher,

Sorry for the very long delay in replying. I am not an expert with the
Vagrant stuff, so I'm adding Angelo in Cc who hopefully will be able to
comment further. Still, I have a few comments.

First, the commit title should be:

	support/misc/Vagrantfile: add libvirt/kvm support

On Fri, 15 Feb 2019 18:36:07 +0100
Christopher Peeters <christopher at lecroq.be> wrote:

> Signed-off-by: Christopher Peeters <christopher at lecroq.be>
> ---
>  support/misc/Vagrantfile | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/support/misc/Vagrantfile b/support/misc/Vagrantfile
> index 1ca5edc647..48f01a5807 100644
> --- a/support/misc/Vagrantfile
> +++ b/support/misc/Vagrantfile
> @@ -29,6 +29,12 @@ Vagrant.configure('2') do |config|
>  		end
>  	end
>  
> +  config.vm.provider :libvirt do |v, override|
> +    override.vm.box = 'generic/ubuntu1804'

Why are you overriding the distribution being used? It seems odd that
depending on the virtualization technology, a different distribution
gets installed inside the VM.

> +    v.memory = VM_MEMORY
> +    v.cpus = VM_CORES
> +  end 

The indentation is also bogus: the file uses tabs, and you used two
spaces. Could you fix that up ?

Thanks,

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com


More information about the buildroot mailing list