+my ($major, $minor, undef) = split(/\./, $git_version);

You don't need trailing undef here. (The number of components is a git version varies between 3 and 4, so you can't make the number of items of the left side always match anyway.)

+               if ($git_unsafe && $git_url !~ /^(?:https?|git|ssh|file):[^:]/) 
{

SSH protocol has an alternative (and I guess more popular) scp-like syntax:

   [user@]example.org:path/to/repo

There are also two syntaxes for local repositories, although I think neither should be allowed. It's *web*checkout after all...

--
Jakub Wilk

Reply via email to