[apparmor] [new directory + files] utils/vim/

Christian Boltz apparmor at cboltz.de
Tue Apr 5 22:27:52 UTC 2011


Hello,

Am Dienstag, 5. April 2011 schrieb Steve Beattie:
> On Mon, Apr 04, 2011 at 05:28:12PM -0700, Kees Cook wrote:
> > On Sat, Apr 02, 2011 at 08:24:59PM +0200, Christian Boltz wrote:
> > > I'm going to create a utils/vim/ directory in trunk and commit
> > > the attached files there. Please review ;-)
> > 
> > Looks okay to me on quick review.
> 
> Yep, go ahead and commit, we can suggest changes once it's in there.
> Thanks!

Done, r1707.

> > We should really hook it up to the "make install" target too,
> > though that's been a standing bug for a while.
> 
> Hooking it to make install and having it be honored by vim is
> trickier than it appears, as vim looks for extensions in a versioned
> subdirectory (e.g. /usr/share/vim/vim73/syntax/ on natty, and no,
> installing to /usr/share/vim/syntax/ doesn't get picked up by vim).

Exactly this is the problem.

For openSUSE, the current solution is that I submit apparmor.vim to the 
vim package manually.

A working automated solution might be:
- package apparmor.vim somewhere in the apparmor-parser package[1], say 
  /usr/share/apparmor/apparmor.vim [2]
- add apparmor-parser to the BuildRequires of vim
- let the vim build copy /usr/share/apparmor/apparmor.vim to
  /usr/share/vim/vim$version/syntax/

The only disadvantage is the additional BuildRequire for vim - we'll see 
if packagers like it or not ;-)


Regards,

Christian Boltz

[1] apparmor-parser already contains the apparmor.vim manpage.
    apparmor-devel would even be better, however there isn't such a
    package yet (at least on openSUSE) and creating it for just one file 
    would result in more metadata than content ;-)

[2] or any other location, this is just a proposal
-- 
postfix.admin developer
Infos and Download: http://postfixadmin.sf.net



More information about the AppArmor mailing list