Re: [PATCH 6/5] t9001: check send-email behavior with implicit sender

2012-11-28 Thread Junio C Hamano
Jeff King writes: > On Wed, Nov 28, 2012 at 03:06:26PM -0500, Jeff King wrote: > >> Here's a cleaned up version that makes it more obvious the commands are >> the same (it also fixes a few minor whitespace problems on the >> indentation, which you can see from the quoting above). > > I wondered h

Re: [PATCH 6/5] t9001: check send-email behavior with implicit sender

2012-11-28 Thread Jeff King
On Wed, Nov 28, 2012 at 03:06:26PM -0500, Jeff King wrote: > Here's a cleaned up version that makes it more obvious the commands are > the same (it also fixes a few minor whitespace problems on the > indentation, which you can see from the quoting above). I wondered how painful it would be to act

Re: [PATCH 6/5] t9001: check send-email behavior with implicit sender

2012-11-28 Thread Jeff King
On Wed, Nov 28, 2012 at 10:55:02AM -0800, Junio C Hamano wrote: > > +test_expect_success $PREREQ,AUTOIDENT 'implicit ident is allowed' ' > > + clean_fake_sendmail && > > + (sane_unset GIT_AUTHOR_NAME && > > + sane_unset GIT_AUTHOR_EMAIL && > > + sane_unset GIT_COMMITTER_NAME && > >

Re: [PATCH 6/5] t9001: check send-email behavior with implicit sender

2012-11-28 Thread Junio C Hamano
Jeff King writes: > On Wed, Nov 28, 2012 at 01:25:35PM -0500, Jeff King wrote: > >> Here are the cleanups and refactorings split out from my >> jk/send-email-sender-prompt series. They can go right on master and are >> independent of Felipe's fc/send-email-no-sender-prompt topic. >> [...] >> Drop

[PATCH 6/5] t9001: check send-email behavior with implicit sender

2012-11-28 Thread Jeff King
On Wed, Nov 28, 2012 at 01:25:35PM -0500, Jeff King wrote: > Here are the cleanups and refactorings split out from my > jk/send-email-sender-prompt series. They can go right on master and are > independent of Felipe's fc/send-email-no-sender-prompt topic. > [...] > Dropped were: > [...] > - send