author | Junio C Hamano <gitster@pobox.com> | |
Wed, 20 Jun 2007 20:47:34 +0000 (13:47 -0700) | ||
committer | Junio C Hamano <gitster@pobox.com> | |
Sun, 24 Jun 2007 09:46:35 +0000 (02:46 -0700) | ||
commit | aeb59328453cd4f438345ea79ff04c96bccbbbb8 | |
tree | 9c8bd0af624beca5703234ad8741aeec39bd0ac5 | tree | snapshot |
parent | 0d351e9ca23ea3be9aa7631b1b548f5d75f01a13 | commit | diff |
git-send-email: Do not make @-less message ID
When the original $from address fails to yield a valid-looking
e-mail address, we created a bogus looking message ID, formatted
like this:
Message-Id: <11823357623688-git-send-email->
This commit fixes it by moving call to make_message_id() to
where it matters, namely, before the $message_id is needed to be
placed in the generated e-mail header; this has an important
side effect of making it clear that $from is already available.
Also throw in Sys::Hostname::hostname() just for fun, although I
suspect that the code would never trigger due to the modified
call sequence that makes sure $from is always available. This
is based on a suggestion by Michael Hendricks.
Signed-off-by: Junio C Hamano <gitster@pobox.com>
When the original $from address fails to yield a valid-looking
e-mail address, we created a bogus looking message ID, formatted
like this:
Message-Id: <11823357623688-git-send-email->
This commit fixes it by moving call to make_message_id() to
where it matters, namely, before the $message_id is needed to be
placed in the generated e-mail header; this has an important
side effect of making it clear that $from is already available.
Also throw in Sys::Hostname::hostname() just for fun, although I
suspect that the code would never trigger due to the modified
call sequence that makes sure $from is always available. This
is based on a suggestion by Michael Hendricks.
Signed-off-by: Junio C Hamano <gitster@pobox.com>
git-send-email.perl | diff | blob | history |