Re: [PATCH] git-svn: remove redundant slashes from show-ignore

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Eric Wong <normalperson@xxxxxxxx> wrote:
> Harvey Harrison <harvey.harrison@xxxxxxxxx> 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
--
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

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux