Richard W.M. Jones
2020-Jan-10 14:59 UTC
[Libguestfs] [PATCH Fedora libguestfs v2] Don't depend on libvirt-daemon-kvm
This gets rid of the various storage drivers. I believe we do not use type=volume, *except* on secrets. Rich.
Richard W.M. Jones
2020-Jan-10 14:59 UTC
[Libguestfs] [PATCH Fedora libguestfs v2] Don't depend on libvirt-daemon-kvm monolith.
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. --- libguestfs.spec | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/libguestfs.spec b/libguestfs.spec index 0d32ef0..872d430 100644 --- a/libguestfs.spec +++ b/libguestfs.spec @@ -211,12 +211,10 @@ 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 -Requires: libvirt-daemon-qemu >= 0.10.2-3 -%endif +# For qemu direct and libvirt backends. +Requires: qemu-kvm-core +Requires: libvirt-daemon-driver-qemu +Requires: libvirt-daemon-driver-secret Recommends: libvirt-daemon-config-network Requires: selinux-policy >= 3.11.1-63 -- 2.24.1
Pino Toscano
2020-Jan-10 16:51 UTC
Re: [Libguestfs] [PATCH Fedora libguestfs v2] Don't depend on libvirt-daemon-kvm monolith.
On Friday, 10 January 2020 15:59:35 CET 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. > --- > libguestfs.spec | 10 ++++------ > 1 file changed, 4 insertions(+), 6 deletions(-) > > diff --git a/libguestfs.spec b/libguestfs.spec > index 0d32ef0..872d430 100644 > --- a/libguestfs.spec > +++ b/libguestfs.spec > @@ -211,12 +211,10 @@ 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 > -Requires: libvirt-daemon-qemu >= 0.10.2-3 > -%endif > +# For qemu direct and libvirt backends. > +Requires: qemu-kvm-core > +Requires: libvirt-daemon-driver-qemu > +Requires: libvirt-daemon-driver-secret > Recommends: libvirt-daemon-config-network > Requires: selinux-policy >= 3.11.1-63As pointed out in the discussion of v1, can you please add libvirt-daemon-driver-storage-core as suggest? Otherwise LGTM -- I'd wait for Dan to take a look at it too, to be double sure we are not missing anything. -- Pino Toscano
Possibly Parallel Threads
- Re: [PATCH Fedora libguestfs v2] Don't depend on libvirt-daemon-kvm monolith.
- [PATCH Fedora libguestfs v4] Don't depend on libvirt-daemon-kvm monolith.
- Re: [PATCH Fedora libguestfs] Don't depend on libvirt-daemon-kvm monolith.
- [PATCH Fedora libguestfs] Don't depend on libvirt-daemon-kvm monolith.
- Re: [PATCH Fedora libguestfs] Don't depend on libvirt-daemon-kvm monolith.