On Sun, Apr 11, 2010 at 7:33 PM, Jakub Narebski <jnareb@xxxxxxxxx> wrote:
> To not break backward compatibility, wouldn't it be better to check if
> elements of @git_base_url_list end with ':' or '/', and join base with
> project path depending on this condition, i.e.:
>
> + @url_list = map { m/[/:]$/ ? "$_$project" : "$_/$project" } @git_base_url_list
> + unless @url_list;
>
> This means: if base ends with colon ':' or slash '/', concatenate base
> and project path, otherwise join them using '/' as field separator.
Thanks for the review. Indeed this way sounds better.
All this is new to me so I'm not sure what is the way forward. Should
I publish another patch or would you?
Damien Tournoud
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html