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

[Martin Kletzander]

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.

>--
>1.8.3.1
>
>--
>libvir-list mailing list
>libvir-list at redhat.com
>https://www.redhat.com/mailman/listinfo/libvir-list
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20171113/f6a8fba4/attachment-0001.sig>