[apparmor] [patch 6/6] add apparmor.vim install target to utils/ install

Christian Boltz apparmor at cboltz.de
Fri Mar 23 00:07:35 UTC 2012


Hello,

Am Donnerstag, 22. März 2012 schrieb Steve Beattie:
> This patch adds a make install target for the generated apparmor.vim
> file, installing by default into /usr/share/apparmor based on IRC
> discussions; alternate suggestions welcome. 

That would be the second apparmor directory containing one file - the 
existing one is /lib/apparmor/ (for rc.apparmor.functions)

OTOH apparmor.vim isn't important enough for /lib, therefore 
/usr/share/apparmor sounds good.

Packaging-wise, it will probably go into the -devel package, so the 
location isn't too relevant for 99% of the users.


BTW: The question reminds me that we are still searching for a better 
location for the extras profiles - maybe /usr/share/apparmor would be a 
good place for them?

> (Installing directly
> into the vim syntax tree is difficult as the system path by default
> contains the vim version number.)

Yes, that's a known problem :-/

The easiest solution is BuildRequires: apparmor-devel in the vim build 
and then copy apparmor.vim to the correct location. I'll see how the 
openSUSE vim maintainer thinks about this idea - probably depends on how 
many other packages apparmor-devel will drag into the build... (For 
additional fun, I have to keep in mind that the vim package needs to 
build for older openSUSE releases.)

> This is kept as a separate patch in case it's felt that it's too late
> in the dev cycle to add an additional installed file, even if it's
> one that won't get automatically used by anything.

It can't break anything ;-)

Acked-By: Christian Boltz <apparmor at cboltz.de>


Regards,

Christian Boltz
-- 
This is like searching for a needle in a haystack in 20km distance
with the naked eye...
[Matthias Hopf in https://bugzilla.novell.com/show_bug.cgi?id=141443]




More information about the AppArmor mailing list