[libvirt PATCH 2/2] meson: Stop looking up ZFS programs at build time
Andrea Bolognani
abologna at redhat.com
Wed Nov 3 10:12:49 UTC 2021
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 <abologna at redhat.com>
---
meson.build | 16 ----------------
src/storage/storage_backend_zfs.c | 3 +++
2 files changed, 3 insertions(+), 16 deletions(-)
diff --git a/meson.build b/meson.build
index 5b73713e50..414e10960d 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 at _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 at _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
--
2.31.1
More information about the libvir-list
mailing list