[libvirt PATCH v3 0/4] spec: Reorganize some packages

[Michal Privoznik]

On 4/20/21 2:30 PM, Andrea Bolognani wrote:
> Upgrading to packages built with these patches applied results in
> 
>    $ sudo dnf update ./*.rpm
>    Dependencies resolved.
>    ===========================================================================
>     Package                                                      Version
>    ===========================================================================
>    Upgrading:
>     libvirt                                                      7.3.0-1.fc32
>     libvirt-client                                               7.3.0-1.fc32
>         replacing  libvirt-bash-completion.x86_64 7.2.0-1.fc32
>     libvirt-daemon                                               7.3.0-1.fc32
>         replacing  libvirt-admin.x86_64 7.2.0-1.fc32
>         replacing  libvirt-bash-completion.x86_64 7.2.0-1.fc32
>     libvirt-daemon-config-network                                7.3.0-1.fc32
>     libvirt-daemon-config-nwfilter                               7.3.0-1.fc32
>     libvirt-daemon-driver-interface                              7.3.0-1.fc32
>     libvirt-daemon-driver-libxl                                  7.3.0-1.fc32
>     libvirt-daemon-driver-lxc                                    7.3.0-1.fc32
>     libvirt-daemon-driver-network                                7.3.0-1.fc32
>     libvirt-daemon-driver-nodedev                                7.3.0-1.fc32
>     libvirt-daemon-driver-nwfilter                               7.3.0-1.fc32
>     libvirt-daemon-driver-qemu                                   7.3.0-1.fc32
>     libvirt-daemon-driver-secret                                 7.3.0-1.fc32
>     libvirt-daemon-driver-storage                                7.3.0-1.fc32
>     libvirt-daemon-driver-storage-core                           7.3.0-1.fc32
>     libvirt-daemon-driver-storage-disk                           7.3.0-1.fc32
>     libvirt-daemon-driver-storage-gluster                        7.3.0-1.fc32
>     libvirt-daemon-driver-storage-iscsi                          7.3.0-1.fc32
>     libvirt-daemon-driver-storage-iscsi-direct                   7.3.0-1.fc32
>     libvirt-daemon-driver-storage-logical                        7.3.0-1.fc32
>     libvirt-daemon-driver-storage-mpath                          7.3.0-1.fc32
>     libvirt-daemon-driver-storage-rbd                            7.3.0-1.fc32
>     libvirt-daemon-driver-storage-scsi                           7.3.0-1.fc32
>     libvirt-daemon-driver-storage-sheepdog                       7.3.0-1.fc32
>     libvirt-daemon-driver-storage-zfs                            7.3.0-1.fc32
>     libvirt-daemon-driver-vbox                                   7.3.0-1.fc32
>     libvirt-daemon-kvm                                           7.3.0-1.fc32
>     libvirt-devel                                                7.3.0-1.fc32
>     libvirt-libs                                                 7.3.0-1.fc32
>     libvirt-nss                                                  7.3.0-1.fc32
>    
>    Transaction Summary
>    ===========================================================================
>    Upgrade  30 Packages
>    
>    Total size: 8.2 M
>    Is this ok [y/N]
> 
> which is exactly what we were aiming for.
> 
> Changes from [v2]:
> 
>    * move virt-admin to -daemon rather than -client;
> 
>    * move other host-only tools from -client to -daemon;
> 
>    * move systemtap probes from -client to -libs.
> 
> Changes from [v1]:
> 
>    * add Obsoletes/Provides for a smooth transition.
> 
> [v2] https://listman.redhat.com/archives/libvir-list/2021-April/msg00613.html
> [v1] https://listman.redhat.com/archives/libvir-list/2021-April/msg00604.html
> 
> Andrea Bolognani (4):
>    spec: Merge -admin package into -daemon
>    spec: Move some files from -client to -daemon
>    spec: Move systemtap probes from -client to -libs
>    spec: Drop -bash-completion package
> 
>   libvirt.spec.in | 74 ++++++++++++++++++++++++-------------------------
>   1 file changed, 36 insertions(+), 38 deletions(-)
> 

A lot of work to have one management install virt-admin (when all they 
could do is to put -admin package onto their list of RPMs to install).

But if you fix all nits I've pointed out then you can merge these:

Reviewed-by: Michal Privoznik <mprivozn at redhat.com>

Michal