[Libguestfs] [PATCH Fedora libguestfs v2] Don't depend on libvirt-daemon-kvm monolith.

Pino Toscano ptoscano at redhat.com
Fri Jan 10 16:51:39 UTC 2020


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-63

As 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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: This is a digitally signed message part.
URL: <http://listman.redhat.com/archives/libguestfs/attachments/20200110/3c893c08/attachment.sig>


More information about the Libguestfs mailing list