[Buildroot] [PATCH v2 1/2] network-manager: Bump to version 1.10.2

Thomas Petazzoni thomas.petazzoni at bootlin.com
Fri Feb 2 10:56:35 UTC 2018


On Fri,  2 Feb 2018 00:45:15 +0100, Petr Vorel wrote:

> +ifeq ($(BR2_PACKAGE_JANSSON),y)
> +NETWORK_MANAGER_CONF_OPTS += --enable-ovs
> +else
> +NETWORK_MANAGER_CONF_OPTS += --disable-ovs
> +endif

It is somewhat weird to introduce this here, and then add an explicit
option in a second patch.

Perhaps PATCH 1/2 could just do --disable-ovs unconditionally, and
PATCH 2/2 add the rest. If you agree, I can rework locally before
applying, I understand it's a lot of nitpicking for such a small
change :)

Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering

More information about the buildroot mailing list