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. 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. -- Felipe Contreras -- 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
