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

On Mon, Jan 09, 2023 at 02:25:44PM +0000, Daniel P. Berrangé wrote: > I still th8ink that dropping the dep is wrong, becasue that is > defeating the purpose of 'libvirt-daemon-$HYPERVISOR' packages > aim todo: > > "installs every possible libvirt feature for the virtualization > driver in question" > > https://libvirt.org/kbase/rpm-deployment.html#full-features-for-one-virt-... > > If you don't want all the features, then don't install that > package, pick the minimal install instead > > https://libvirt.org/kbase/rpm-deployment.html#minimal-features-for-one-vi... > >> If you feel that the various libvirt-daemon-$hypervisor packages need >> to keep providing the same exact functionality over time, then yes, >> we can't drop or downgrade any of the dependencies. But, at the same >> time, we're potentially breaking consumers by taking away libvirtd... > > Where the functionality still exists, the packages should continue > to provide it. The case of libvirtd is a bit special, as we're in > the process of phasing out its existance. That's not the cae for > virtlockd. Okay, the fact that we're explicitly documenting these packages as the "every possible feature for the driver" option means that I have no choice but to agree with you :) Jim, can you please respin while reverting these changes that I had suggested? Sorry :(