Re: [libvirt] [PATCH] spec: don't package product dirs

On Thu, Sep 28, 2017 at 09:35:45AM +0300, Nikolay Shirokovskiy wrote: > Directories /var/{lib,cache}/libvirt/qemu/ are created by libvirtd on > start and their owner:group is changed according to the config. Thus > no need to include them in libvirt-daemon-driver-qemu package. Otherwise > we see noisy "directory changed" on rpm -V for the package. > --- > libvirt.spec.in | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/libvirt.spec.in b/libvirt.spec.in > index a3bd77f..e20f65c 100644 > --- a/libvirt.spec.in > +++ b/libvirt.spec.in > @@ -1911,8 +1911,6 @@ exit 0 > %config(noreplace) %{_sysconfdir}/libvirt/qemu-lockd.conf > %config(noreplace) %{_sysconfdir}/logrotate.d/libvirtd.qemu > %ghost %dir %attr(0700, root, root) %{_localstatedir}/run/libvirt/qemu/ > -%dir %attr(0751, %{qemu_user}, %{qemu_group}) %{_localstatedir}/lib/libvirt/qemu/ > -%dir %attr(0750, %{qemu_user}, %{qemu_group}) %{_localstatedir}/cache/libvirt/qemu/ > %{_datadir}/augeas/lenses/libvirtd_qemu.aug > %{_datadir}/augeas/lenses/tests/test_libvirtd_qemu.aug > %{_libdir}/%{name}/connection-driver/libvirt_driver_qemu.so I agree, however I think this should be done on all subdirectories of %{_localstatedir}/{cache,lib}/libvirt/.  That directories are owned by libvirt-daemon and libvirt-libs respectively, so it should be OK.