Re: [libvirt] [PATCH 1/2] Teach AppArmor, that /usr/lib64 may exist.

2015-01-05 Thread Eric Blake
On 12/30/2014 03:33 AM, Cédric Bosdonnat wrote: s/,// in the subject Also, we tend to avoid trailing '.' in commit summary lines, although that is not strictly enforced > The apparmor profiles forgot about /usr/lib64 folders, just add lib64 > as a possible alternative to lib in the paths > --- >

Re: [libvirt] [PATCH 1/2] Teach AppArmor, that /usr/lib64 may exist.

2015-01-05 Thread Cedric Bosdonnat
On Sun, 2015-01-04 at 10:00 -0600, Jamie Strandboge wrote: > On 12/30/2014 04:33 AM, Cédric Bosdonnat wrote: > > The apparmor profiles forgot about /usr/lib64 folders, just add lib64 > > as a possible alternative to lib in the paths > > These changes all look good to me. +1 Pushed, then. Thanks f

Re: [libvirt] [PATCH 1/2] Teach AppArmor, that /usr/lib64 may exist.

2015-01-04 Thread Jamie Strandboge
On 12/30/2014 04:33 AM, Cédric Bosdonnat wrote: > The apparmor profiles forgot about /usr/lib64 folders, just add lib64 > as a possible alternative to lib in the paths These changes all look good to me. +1 > --- > examples/apparmor/libvirt-qemu | 2 +- > examples/apparmor/usr.l

[libvirt] [PATCH 1/2] Teach AppArmor, that /usr/lib64 may exist.

2014-12-30 Thread Cédric Bosdonnat
The apparmor profiles forgot about /usr/lib64 folders, just add lib64 as a possible alternative to lib in the paths --- examples/apparmor/libvirt-qemu | 2 +- examples/apparmor/usr.lib.libvirt.virt-aa-helper | 4 ++-- examples/apparmor/usr.sbin.libvirtd | 4 ++-- 3 f