Re: [PATCH V5 10/11] spec: Remove libvirt-daemon dependency from hypervisor subpackages

On Fri, Dec 23, 2022 at 11:56:03AM -0700, Jim Fehlig wrote: > %package daemon-qemu > Summary: Server side daemon & driver required to run QEMU guests > > + %if %{with_modular_daemons} > +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} > + %else > Requires: libvirt-daemon = %{version}-%{release} > + %endif This version is fine, but as explained elsewhere I think it would be better to have Requires: libvirt-daemon-common = %{version}-%{release} Requires: libvirt-daemon-log = %{version}-%{release} Recommends: libvirt-daemon-proxy = %{version}-%{release} and no dependency at all on the locking part. Rationale: * virtproxyd being present allows clients that are older than ~2 years to connect, so it should be there by default while still making it possible for the admin to opt out, which can be done by simply uninstalling the corresponding package;

* storage locking is not the default behavior and needs to be turned on explicitly, so it's not a big deal if part of the setup involves installing a couple extra packages in addition to editing some configuration files, and everyone else gets a leaner installation.