diff mbox

[2/4] vde2: enable building host package

Message ID 1467584664-11640-3-git-send-email-thomas.petazzoni@free-electrons.com
State Accepted
Commit 692e392448255fe5dab824072a0a6e01977f7087
Headers show

Commit Message

Thomas Petazzoni July 3, 2016, 10:24 p.m. UTC
From: Simon Maes <simonn.maes@gmail.com>

In preparation of enabling vde2 support in host-qemu package.

Signed-off-by: Simon Maes <simonn.maes@gmail.com>
[Thomas:
 - add HOST_VDE2_CONF_OPTS and HOST_VDE2_MAKE, like we have for the
   target variant.]
Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 package/vde2/vde2.mk | 12 ++++++++++++
 1 file changed, 12 insertions(+)

Comments

Yann E. MORIN July 3, 2016, 11:18 p.m. UTC | #1
Simon, Thomas, All,

On 2016-07-04 00:24 +0200, Thomas Petazzoni spake thusly:
> From: Simon Maes <simonn.maes@gmail.com>
> 
> In preparation of enabling vde2 support in host-qemu package.

I'm not sure why we need to have a host-qemu with VDE2 support...

As far as I understand it, VDE2 is needed for complex and complicated
cloud-like networking infrastructure, whereas we provide a host-qemu
that will serve to quickly test a Buildroot build.

I think ew should not add such a dependency.

However, I'll still do a review...

> Signed-off-by: Simon Maes <simonn.maes@gmail.com>
> [Thomas:
>  - add HOST_VDE2_CONF_OPTS and HOST_VDE2_MAKE, like we have for the
>    target variant.]
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> ---
>  package/vde2/vde2.mk | 12 ++++++++++++
>  1 file changed, 12 insertions(+)
> 
> diff --git a/package/vde2/vde2.mk b/package/vde2/vde2.mk
> index bad758d..d9b11bb 100644
> --- a/package/vde2/vde2.mk
> +++ b/package/vde2/vde2.mk
> @@ -37,4 +37,16 @@ VDE2_CONF_OPTS = \
>  # Package does not build in parallel due to improper make rules
    ^
Non-breakable space here...
Note: there's another one for the first item in the list, above...

Probably my deed, btw...

>  VDE2_MAKE = $(MAKE1)
>  
> +HOST_VDE2_CONF_OPTS = \
> +	--disable-experimental  \
> +	--disable-cryptcab      \
> +	--disable-pcap          \
> +	--disable-python        \
> +	--disable-profile       \
> +	--disable-kernel-switch \
> +	--enable-tuntap         \
> +
> +HOST_VDE2_MAKE = $(MAKE1)
> +
>  $(eval $(autotools-package))
> +$(eval $(host-autotools-package))

    Reviewed-by: "Yann E. MORIN" <yann.morin.1998@free.fr>

Regards,
Yann E. MORIN.

> -- 
> 2.7.4
> 
> _______________________________________________
> buildroot mailing list
> buildroot@busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
diff mbox

Patch

diff --git a/package/vde2/vde2.mk b/package/vde2/vde2.mk
index bad758d..d9b11bb 100644
--- a/package/vde2/vde2.mk
+++ b/package/vde2/vde2.mk
@@ -37,4 +37,16 @@  VDE2_CONF_OPTS = \
 # Package does not build in parallel due to improper make rules
 VDE2_MAKE = $(MAKE1)
 
+HOST_VDE2_CONF_OPTS = \
+	--disable-experimental  \
+	--disable-cryptcab      \
+	--disable-pcap          \
+	--disable-python        \
+	--disable-profile       \
+	--disable-kernel-switch \
+	--enable-tuntap         \
+
+HOST_VDE2_MAKE = $(MAKE1)
+
 $(eval $(autotools-package))
+$(eval $(host-autotools-package))