ciaranm / securemodelines

A secure alternative to Vim modelines

107 stars 29 forks source link

Modeline not recognized with fully patched vim #7

Closed jmcantrell closed 12 years ago

jmcantrell commented 12 years ago

Vim :version

VIM - Vi IMproved 7.3 (2010 Aug 15, compiled Oct 21 2011 12:06:38) Included patches: 1-346

Modeline not recognized

vim:set ft=sh:

If it helps, the modeline is at the end of the file. I haven't tried it on other systems, but this is on Ubuntu Oneiric.

ciaranm commented 12 years ago

Unless you are running a recent version of the plugin, you'll need to put spaces around the colons.

jmcantrell commented 12 years ago

I'm pulling it directly from this repo. Also, I put the same modeline at the top of the file, and it recognized it then.

tgray commented 12 years ago

The most recent commit to this plugin broke it for me as well. I can't get it to recognize anything. I had to revert back to ciaranm/securemodelines@1b76cb6a567c20f3f51949ac1d4672911d108054.

ciaranm commented 12 years ago

Ah, think I've fixed it.

I was getting confused by having two different versions of the thing installed at different places. Oops.