[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-email

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

2012-11-28 Thread Junio C Hamano
Jeff King p...@peff.net 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. [...]

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 + sane_unset

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

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

2012-11-28 Thread Junio C Hamano
Jeff King p...@peff.net 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