Merging of hinrik and jettero forks

11 views
Skip to first unread message

Andy Lester

unread,
Jul 4, 2009, 2:14:05 PM7/4/09
to vim-...@googlegroups.com
I've merged hinrik's and jettero's forks to the main petdance/vim-
perl, modulo some minor changes along the way, specifically:

* Not recognizing ^G as a q() character
* Updates to the header blocks
* Removing commented-out code.

hinrik has ftplugin directory in his fork, but I didn't merge that
in. Should ftplugin/perl6?.vim be part of this project?

Once we make sure all is well with the merging, I'll forward files off
to Bram.

xoxo,
Andy


--
Andy Lester => an...@petdance.com => www.theworkinggeek.com => AIM:petdance


Hinrik Örn Sigurðsson

unread,
Jul 5, 2009, 5:56:59 PM7/5/09
to vim-...@googlegroups.com
Yeah, ftplugin/perl6.vim is useful. It defines various things like
Perl 6 search path (@*INC, basically), which words constitute an
include ('use' and 'require' currently), and in the future it will
specify a keyword program, which is perldoc in the case of Perl 5.
There are some other things in there which are similar or identical to
the Perl 5 version, but we really should keep the two separate.

I think it would be good to hold off sending patches to Bram until we
have patched the filetype-detection in vim. It is currently almost
impossible to make vim detect a file as Perl 6 without first deciding
that it's Perl 5. This means that it always executes ftplugin/perl.vim
(and indent/perl.vim too if I'm not mistaken) for Perl 6 files if they
have a .pl extension. The file that needs to be modified is
$VIMRUNTIME/filetype.vim if I recall. It will be good to have Perl 5
and Perl 6 completely separate in vim.

Andy Lester

unread,
Jul 6, 2009, 1:00:36 AM7/6/09
to vim-...@googlegroups.com

On Jul 5, 2009, at 4:56 PM, Hinrik Örn Sigurðsson wrote:

> It will be good to have Perl 5
> and Perl 6 completely separate in vim.


That's one of the reasons I took this on, so that nobody would try to
have perl.vim somehow try to overstep its capabilities and handle both.

ftplugin/perl6.vim is now commited + pushed.

Reply all
Reply to author
Forward
0 new messages