From: Eric Wong <normalperson@yhbt.net>
Date: Mon, 31 Mar 2008 06:26:37 GMT
Subject: Re: [PATCH] git-svn: remove redundant slashes from show-ignore
Message-ID: <20080331062637.GA3082@hand.yhbt.net>
In-Reply-To: <20080330013600.GB16157@soma>

Eric Wong <normalperson@yhbt.net> wrote:
> Harvey Harrison <harvey.harrison@gmail.com> wrote:
> > On Sat, 2008-03-29 at 16:37 -0700, Eric Wong wrote:
> > > Jonathan Scott Duff wrote:
> > > > Recently I tried "git svn showignore" on my parrot repository and it
> > > > failed.  I tracked it down to the prop_walk() sub.  When it recurses,
> > > > $path has an extra / on the beginning (i.e., when it recurses, it
> > > > tries to get the props for "//apps" instead of "/apps").   I *think*
> > > > this is because $path is used in the recursive call rather than $p
> > > > (which seems to contain a properly transformed $path).  Anyway, I've
> > > > attached a patch that works for me and I think is generally the right
> > > > thing.
> > > 
> > > Patch-submitted-by: Jonathan Scott Duff
> > 
> > Shouldn't that be Signed-off-by?
> 
> I'm not sure what the proper protocol for this should be,
> as Jonathan emailed the patch to me privately without a sign-off...
> 
> [1] - Out of respect for his privacy, I'm only Bcc-ing him to avoid
>   putting his address on the list.

Signed-off-by: Jonathan Scott Duff

Jonathan Scott Duff wrote:
> Er ... yeah.  I've very new to git (long time svn user though) so I don't
> know what the cultural norms are here.  So if I need to "sign off" on the
> patch, consider it signed off :-)

-- 
Eric Wong