On 2013-11-11 14:29, Felipe Contreras wrote: > On Mon, Nov 11, 2013 at 1:19 PM, Richard Hansen <[email protected]> wrote: >> On 2013-11-11 06:42, Felipe Contreras wrote: >>> Richard Hansen wrote: >>>> It's hard to tell which author conversion test failed when the email >>>> addresses look similar. >>>> >>>> Signed-off-by: Richard Hansen <[email protected]> >>>> --- >>>> contrib/remote-helpers/test-hg.sh | 20 ++++++++++---------- >>>> 1 file changed, 10 insertions(+), 10 deletions(-) >>>> >>>> diff --git a/contrib/remote-helpers/test-hg.sh >>>> b/contrib/remote-helpers/test-hg.sh >>>> index 84c67ff..5eda265 100755 >>>> --- a/contrib/remote-helpers/test-hg.sh >>>> +++ b/contrib/remote-helpers/test-hg.sh >>>> @@ -209,16 +209,16 @@ test_expect_success 'authors' ' >>>> >>>> >../expected && >>>> author_test alpha "" "H G Wells <[email protected]>" && >>>> - author_test beta "test" "test <unknown>" && >>>> - author_test beta "test <[email protected]> (comment)" "test >>>> <[email protected]>" && >> >> Notice the two betas here in the original code. > > Ahh, that's a bug. > >>>> - author_test gamma "<[email protected]>" "Unknown <[email protected]>" && >>>> - author_test delta "name<[email protected]>" "name <[email protected]>" >>>> && >>>> - author_test epsilon "name <[email protected]" "name >>>> <[email protected]>" && >>>> - author_test zeta " test " "test <unknown>" && >>>> - author_test eta "test < [email protected] >" "test <[email protected]>" >>>> && >>>> - author_test theta "test >[email protected]>" "test <[email protected]>" >>>> && >>>> - author_test iota "test < test <at> example <dot> com>" "test >>>> <unknown>" && >>>> - author_test kappa "[email protected]" "Unknown <[email protected]>" >>>> + author_test beta "beta" "beta <unknown>" && >>>> + author_test beta "beta <[email protected]> (comment)" "beta >>>> <[email protected]>" && >>> >>> Two betas? >> >> See above. I can change them to beta1 and beta2, or if you'd prefer I >> can change them to beta and gamma and increment the subsequent entries. > > Yeah, I would prefer that in two patches, one that fixes the sequence, > and the other one that changes the emails.
Will do. Thanks, Richard > If you don't have time for > that the original patch is OK by me. The problem with the sequence can > be fixed later. > > Cheers. > -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
