Re: [PATCH] mailmap: Testing the single letter name case.

2013-07-15 Thread Junio C Hamano
Thanks. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH] mailmap: Testing the single letter name case.

2013-07-14 Thread Eric Sunshine
On Sat, Jul 13, 2013 at 4:20 PM, Junio C Hamano wrote: > Stefan Beller writes: > >> Indeed the patch tests for both bugs unintentionally. > > I was puzzled because I do not think that is what is happening with > the posted patch. > > The off-by-one fix seems to be correct from code inspection, bu

Re: [PATCH] mailmap: Testing the single letter name case.

2013-07-13 Thread Eric Sunshine
On Sat, Jul 13, 2013 at 3:20 AM, Stefan Beller wrote: > This is a regression test for a66e77eab70a08938fdc2227b7ada0f0465c6991 > > Signed-off-by: Stefan Beller > --- > t/t4203-mailmap.sh | 41 + > 1 file changed, 41 insertions(+) > > diff --git a/t/t4203-m

Re: [PATCH] mailmap: Testing the single letter name case.

2013-07-13 Thread Junio C Hamano
Stefan Beller writes: > Indeed the patch tests for both bugs unintentionally. I was puzzled because I do not think that is what is happening with the posted patch. If I drop the tip one from jc/mailmap-case-insensitivity and apply this patch instead, the test passes (which is good). gi

Re: [PATCH] mailmap: Testing the single letter name case.

2013-07-13 Thread Stefan Beller
On 07/13/2013 07:38 PM, Junio C Hamano wrote: > Stefan Beller writes: > >> This is a regression test for a66e77eab70a08938fdc2227b7ada0f0465c6991 > > Sorry, I do not quite get this. > > If you apply the patch on top of the said commit before that commit, the > new test does not pass. > > But i

Re: [PATCH] mailmap: Testing the single letter name case.

2013-07-13 Thread Junio C Hamano
Stefan Beller writes: > This is a regression test for a66e77eab70a08938fdc2227b7ada0f0465c6991 Sorry, I do not quite get this. If you apply the patch on top of the said commit before that commit, the new test does not pass. But if you apply the patch on top of the said commit, the new test sti

[PATCH] mailmap: Testing the single letter name case.

2013-07-13 Thread Stefan Beller
This is a regression test for a66e77eab70a08938fdc2227b7ada0f0465c6991 Signed-off-by: Stefan Beller --- t/t4203-mailmap.sh | 41 + 1 file changed, 41 insertions(+) diff --git a/t/t4203-mailmap.sh b/t/t4203-mailmap.sh index 842b754..9ec87a2 100755 --- a/t/