On Fri, May 10, 2013 at 10:29 AM, Ramkumar Ramachandra <artagnon@xxxxxxxxx> wrote: > test-lib.sh already sets a sane GIT_AUTHOR_{NAME,EMAIL} for all test > scripts to use. Don't unnecessarily duplicate the work. > > Signed-off-by: Ramkumar Ramachandra <artagnon@xxxxxxxxx> > --- > t/t3400-rebase.sh | 4 ---- > 1 file changed, 4 deletions(-) > > diff --git a/t/t3400-rebase.sh b/t/t3400-rebase.sh > index b58fa1a..a7ca2f1 100755 > --- a/t/t3400-rebase.sh > +++ b/t/t3400-rebase.sh > @@ -10,10 +10,6 @@ among other things. > ' > . ./test-lib.sh > > -GIT_AUTHOR_NAME=author@name > -GIT_AUTHOR_EMAIL=bogus@email@address These values are intentionally bogus. Doesn't this change defeat the purpose of 43c2325 (am: use get_author_ident_from_commit instead of mailinfo when rebasing; 2010-06-16)? > -export GIT_AUTHOR_NAME GIT_AUTHOR_EMAIL > - > test_expect_success 'prepare repository with topic branches' ' > git config core.logAllRefUpdates true && > echo First >A && -- 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