[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