From: Andrea Bolognani Date: Wed, 3 Nov 2021 09:32:14 +0000 (+0100) Subject: meson: Stop looking up ZFS programs at build time X-Git-Tag: v7.10.0-rc1~209 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=506c3a39d6e645c8414c278ceaba97935f90cb95;p=thirdparty%2Flibvirt.git meson: Stop looking up ZFS programs at build time At this point, we're no longer using the availability of the ZFS programs at build time to decide whether to enable ZFS support, so the only purpose of these find_program() calls is to record their absolute paths. However, the virCommand facilities that we're ultimately using to run them are already capable of performing this lookup at runtime, and in fact that's exactly what we already do in the case of, for example, vstorage. Drop the build time lookups and always perform them at runtime. Signed-off-by: Andrea Bolognani Reviewed-by: Tim Wiederhake --- diff --git a/meson.build b/meson.build index 60bf2a1d72..dd46572365 100644 --- a/meson.build +++ b/meson.build @@ -1975,24 +1975,8 @@ if conf.has('WITH_LIBVIRTD') endif if not get_option('storage_zfs').disabled() - foreach name : [ 'zfs', 'zpool' ] - set_variable( - '@0@_prog'.format(name), - find_program(name, required: false, dirs: libvirt_sbin_path) - ) - endforeach - use_storage = true conf.set('WITH_STORAGE_ZFS', 1) - foreach name : [ 'zfs', 'zpool' ] - prog_var = get_variable('@0@_prog'.format(name)) - if prog_var.found() - prog_path = prog_var.path() - else - prog_path = name - endif - conf.set_quoted(name.to_upper(), prog_path) - endforeach endif endif diff --git a/src/storage/storage_backend_zfs.c b/src/storage/storage_backend_zfs.c index 6fabeade11..2a5d74357d 100644 --- a/src/storage/storage_backend_zfs.c +++ b/src/storage/storage_backend_zfs.c @@ -33,6 +33,9 @@ VIR_LOG_INIT("storage.storage_backend_zfs"); +#define ZFS "zfs" +#define ZPOOL "zpool" + /* * Some common flags of zfs and zpool commands we use: * -H -- don't print headers and separate fields by tab