On Mon, 2017-05-15 at 09:28 -0500, Jamie Strandboge wrote: > On Mon, 2017-05-15 at 15:23 +0200, Stefan Bader wrote: > > From: Felix Geyer <fgeyer@xxxxxxxxxx> > > > > Local overrides is a feature Debian/Ubuntu libvirt provided for a while. > > This allows the user to have a non-conffile that he can use to extend the > > package delivered rules with extra content matching his special case. > > > > This change adds the include directives to the apparmor profiles > > for virt-aa-helper and libvirtd. > > > > I'm fine with this change but it is important to understand that > /etc/apparmor.d/local/usr.sbin.libvirtd must exist otherwise the profile will > fail to load. In Debian/Ubuntu we use dh_apparmor which takes care of this for > us. If this is upstreamed, then wherever install of the profile happens or is > documented, then the local changes file needs to also be installed/documented. > Other non-deb distributions might not like this extra file, so it is possible > this may be a Debian and its derivatives thing.... > Oh heh, I see you adjusted the Makefile.am for this in 08. Thanks! -- Jamie Strandboge | http://www.canonical.com
Attachment:
signature.asc
Description: This is a digitally signed message part
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list