On Fri, Jan 10, 2020 at 02:05:33PM +0000, Richard W.M. Jones wrote:
libguestfs usually needs qemu. However it only requires an emulator
for the same architecture, not for all architectures.
libvirt-daemon-kvm pulls in qemu which pulls in emulators for all
architectures, as well as a bunch of other stuff we don't need at all
like network interface support and nwfilter.
There are no Fedora TCG-only arches, so drop the conditional section.
I also made support for storage backends "Recommends" so that people
can get rid of things like Gluster if they want.
---
libguestfs.spec | 14 +++++++++-----
1 file changed, 9 insertions(+), 5 deletions(-)
diff --git a/libguestfs.spec b/libguestfs.spec
index 0d32ef0..71f97c6 100644
--- a/libguestfs.spec
+++ b/libguestfs.spec
@@ -211,12 +211,16 @@ Requires: fuse
# For core disk-create API.
Requires: /usr/bin/qemu-img
-# For libvirt backend.
-%ifarch %{ix86} x86_64 %{arm} aarch64
-Requires: libvirt-daemon-kvm >= 0.10.2-3
-%else
+# For qemu direct and libvirt backends.
+Requires: qemu-kvm-core
+Requires: libvirt-daemon-driver-qemu
Do you use the libvirt "secret" APIs at all (disk encryption, network
disk auth passwords) ? If so you will need "libvirt-daemon-driver-secret"
too. How about any other libvirt sub-driver APIs ? Networking ? Host
dev, etc ?
+Recommends: libvirt-daemon-driver-storage-disk
+Recommends: libvirt-daemon-driver-storage-gluster
+Recommends: libvirt-daemon-driver-storage-iscsi
+Recommends: libvirt-daemon-driver-storage-logical
+Recommends: libvirt-daemon-driver-storage-rbd
+Recommends: libvirt-daemon-driver-storage-scsi
Requires: libvirt-daemon-qemu >= 0.10.2-3
You need to cull this last existing line too
-%endif
Recommends: libvirt-daemon-config-network
Requires: selinux-policy >= 3.11.1-63
Reviewed-by: Daniel P. Berrangé <berrange(a)redhat.com>
with the line culled
Regards,
Daniel
--
|:
https://berrange.com -o-
https://www.flickr.com/photos/dberrange :|
|:
https://libvirt.org -o-
https://fstop138.berrange.com :|
|:
https://entangle-photo.org -o-
https://www.instagram.com/dberrange :|