Daniel P. Berrange wrote:
On Tue, Dec 03, 2013 at 03:30:08PM -0700, Jim Fehlig wrote:
> The libvirt-daemon package contains several hypervisor-specific files,
> directories, and script, which can be problematic when building the
> package with multiple hypervisor support, e.g. both QEMU and Xen.
>
> E.g. installing a QEMU+Xen enabled libvirt-daemon on a Xen-only system
> will result in the creation of qemu and kvm groups and a qemu user.
>
> Move the hypervisor-specific files, directories, and script to the
> respective hypervisor driver subpackages.
> ---
> libvirt.spec.in | 117 ++++++++++++++++++++++----------------------------------
>
Mostly looks good, but how about the dirs related to the libvirt-daemon-driver-network
RPM - they're still in the main RPM and could be moved too.
While doing that, noticed that all the libvirt-daemon-driver-* are
conditional on 'with_driver_modules'. So it seems these files,
directories, etc are needed in the main RPM when not building driver
modules, but in the subpackages when building driver modules? Hmm, not
sure if that is correct...
Regards,
Jim