meson: Improve network driver enablement logic

The Homebrew package explicitly enables this driver despite us
disabling it by default on macOS, so it must be functional to
at least some extent and certainly can't be causing any build
failures.

Additionally, if the user has explicitly asked for the network
driver to be enabled but libvirtd is disabled for whatever
reason, we should error out instead of silently disabling the
network driver.

Signed-off-by: Andrea Bolognani <abologna@redhat.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com>
This commit is contained in:
Andrea Bolognani 2021-12-03 14:06:54 +01:00
parent 4cf9f43f1d
commit c71181b666

View File

@ -1645,10 +1645,10 @@ if not get_option('driver_ch').disabled() and host_machine.system() == 'linux' a
endif endif
endif endif
# there's no use compiling the network driver without the libvirt if not get_option('driver_network').disabled() and conf.has('WITH_LIBVIRTD')
# daemon, nor compiling it for macOS, where it breaks the compile
if not get_option('driver_network').disabled() and conf.has('WITH_LIBVIRTD') and host_machine.system() != 'darwin'
conf.set('WITH_NETWORK', 1) conf.set('WITH_NETWORK', 1)
elif get_option('driver_network').enabled()
error('libvirtd must be enabled to build the network driver')
endif endif
if udev_dep.found() and conf.has('WITH_LIBVIRTD') if udev_dep.found() and conf.has('WITH_LIBVIRTD')