Hi, On Sun, 27 Jan 2008, Shawn Bohrer wrote: > base + current already includes the trailing slash so adding > one removes the first character of the next directory. > > Signed-off-by: Shawn Bohrer <shawn.bohrer@xxxxxxxxx> > --- > > This fixes the oops part of the issue Johannes found, have I? > but doesn't address the fact that we probably should remove files that > aren't a part of the repository at in the first place. I am sorry, but I cannot begin to see what this commit tries to accomplish. Yes, sure, there is an off-by-one error, and your commit message says how that was fixed. But I miss a description what usage it would affect, i.e. when this bug triggers. I imagine that you would be as lost as me, reading that commit message 6 months from now, trying to understand why that change was made. Ciao, Dscho - 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