Re: [PATCH V2 8/9] spec: Remove libvirt-daemon dependency from hypervisor subpackages
On Wed, Dec 14, 2022 at 09:59:41AM -0800, Andrea Bolognani wrote:
On Tue, Dec 13, 2022 at 05:31:01PM -0700, Jim Fehlig wrote:
> %package daemon-qemu
> Summary: Server side daemon & driver required to run QEMU guests
>
> -Requires: libvirt-daemon = %{version}-%{release}
> +Requires: libvirt-daemon-common = %{version}-%{release}
> +Requires: libvirt-daemon-log = %{version}-%{release}
> +Requires: libvirt-daemon-lock = %{version}-%{release}
> +Requires: libvirt-daemon-plugin-lockd = %{version}-%{release}
> +Requires: libvirt-daemon-plugin-sanlock = %{version}-%{release}
The sanlock plugin was an optional install before, but now it's a
hard dependency. I'm not sure we want that.
Even if we decide that we do, we should only have this dependency if
%{with_sanlock}, otherwise the package will become impossible to
install in certain cases.
Also we need to be more careful here: we want the more granular
dependencies if %{with_modular_daemons}, but otherwise (e.g. RHEL 8)
we still want the monolithic daemon.
--
Andrea Bolognani / Red Hat / Virtualization