Re: [PATCH] gitweb: Remove extra "/" in path names for git_get_project_list

From: Jakub Narebski <jnareb@gmail.com>
Date: Nov 3, 2006, 11:18 UTC
Message-ID: <eif8gd$6u5$1@sea.gmane.org>
In-Reply-To: <7vslh1jcji.fsf@assigned-by-dhcp.cox.net>

Junio C Hamano wrote:

Show 14 quoted lines

> "Aneesh Kumar K.V" <aneesh.kumar@gmail.com> writes:
> 
>> Without this change we get a wrong $pfxlen value and the check_export_ok()
>> checks with with a wrong directory name. Without this patch the below
>> $projects_list fails with gitweb
>>
>> $projects_list = "/tmp/a/b/";
>>
>> Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@gmail.com>
> 
> Hmph.  Doesn't this break $projects_list = "/", I wonder?
> 
>> +            # remove the trailing "/"
>> +            $dir =~ s!/+$!!;

So perhaps

        $dir =~ s!(?<=[^/])/+$!!;

(zero-width positive look-behind assertion).

-- 
Jakub Narebski
Warsaw, Poland
ShadeHawk on #git