[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