[PATCH V9 11/14] spec: Remove libvirt-daemon dependency from primary drivers

Jim Fehlig posted 14 patches 3 years ago
[PATCH V9 11/14] spec: Remove libvirt-daemon dependency from primary drivers
Posted by Jim Fehlig 3 years ago
To avoid needlessly installing the monolithic daemon, replace the
libvirt-daemon dependency with libvirt-daemon-common in the primary
drivers.

The qemu driver also needs a dependency on libvirt-daemon-log since
the virtqemud systemd service file has a hard dependency on
virtlogd.socket.

Signed-off-by: Jim Fehlig <jfehlig@suse.com>
---
 libvirt.spec.in | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/libvirt.spec.in b/libvirt.spec.in
index 4886b91bf8..5e769ee32f 100644
--- a/libvirt.spec.in
+++ b/libvirt.spec.in
@@ -749,7 +749,8 @@ parted and more.
 %if %{with_qemu}
 %package daemon-driver-qemu
 Summary: QEMU driver plugin for the libvirtd daemon
-Requires: libvirt-daemon = %{version}-%{release}
+Requires: libvirt-daemon-common = %{version}-%{release}
+Requires: libvirt-daemon-log = %{version}-%{release}
 Requires: libvirt-libs = %{version}-%{release}
 Requires: /usr/bin/qemu-img
 # For image compression
@@ -775,7 +776,7 @@ QEMU
 %if %{with_lxc}
 %package daemon-driver-lxc
 Summary: LXC driver plugin for the libvirtd daemon
-Requires: libvirt-daemon = %{version}-%{release}
+Requires: libvirt-daemon-common = %{version}-%{release}
 Requires: libvirt-libs = %{version}-%{release}
 # There really is a hard cross-driver dependency here
 Requires: libvirt-daemon-driver-network = %{version}-%{release}
@@ -795,7 +796,7 @@ the Linux kernel
 %if %{with_vbox}
 %package daemon-driver-vbox
 Summary: VirtualBox driver plugin for the libvirtd daemon
-Requires: libvirt-daemon = %{version}-%{release}
+Requires: libvirt-daemon-common = %{version}-%{release}
 Requires: libvirt-libs = %{version}-%{release}
 
 %description daemon-driver-vbox
@@ -807,7 +808,7 @@ VirtualBox
 %if %{with_libxl}
 %package daemon-driver-libxl
 Summary: Libxl driver plugin for the libvirtd daemon
-Requires: libvirt-daemon = %{version}-%{release}
+Requires: libvirt-daemon-common = %{version}-%{release}
 Requires: libvirt-libs = %{version}-%{release}
 Obsoletes: libvirt-daemon-driver-xen < 4.3.0
 
-- 
2.38.1
Re: [PATCH V9 11/14] spec: Remove libvirt-daemon dependency from primary drivers
Posted by Daniel P. Berrangé 3 years ago
On Fri, Jan 13, 2023 at 12:21:58PM -0700, Jim Fehlig wrote:
> To avoid needlessly installing the monolithic daemon, replace the
> libvirt-daemon dependency with libvirt-daemon-common in the primary
> drivers.
> 
> The qemu driver also needs a dependency on libvirt-daemon-log since
> the virtqemud systemd service file has a hard dependency on
> virtlogd.socket.
> 
> Signed-off-by: Jim Fehlig <jfehlig@suse.com>
> ---
>  libvirt.spec.in | 9 +++++----
>  1 file changed, 5 insertions(+), 4 deletions(-)

Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>


With 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 :|

Re: [PATCH V9 11/14] spec: Remove libvirt-daemon dependency from primary drivers
Posted by Andrea Bolognani 3 years ago
On Fri, Jan 13, 2023 at 12:21:58PM -0700, Jim Fehlig wrote:
> To avoid needlessly installing the monolithic daemon, replace the
> libvirt-daemon dependency with libvirt-daemon-common in the primary
> drivers.
>
> The qemu driver also needs a dependency on libvirt-daemon-log since
> the virtqemud systemd service file has a hard dependency on
> virtlogd.socket.
>
> Signed-off-by: Jim Fehlig <jfehlig@suse.com>
> ---
>  libvirt.spec.in | 9 +++++----
>  1 file changed, 5 insertions(+), 4 deletions(-)

Reviewed-by: Andrea Bolognani <abologna@redhat.com>

-- 
Andrea Bolognani / Red Hat / Virtualization