Re: [PATCH] git-send-email.perl: fix In-Reply-To for second and subsequent patches
From: Jonathan Nieder <hidden>
Date: 2016-06-15 22:49:47
(+cc: some send-email people) Hi, Antonio Ospite wrote:
Make second and subsequent patches appear as replies to the first patch, even when an initial In-Reply-To is supplied
[...]
Signed-off-by: Antonio Ospite <redacted>
Thanks.
- When $initial_reply_to is asked to the user, it is asked as the
"Message-ID to be used as In-Reply-To for the _first_ email", this
makes me think that the second and subsequent patches are not using
itThis kind of justification belongs in the commit message, no? That way, we can save future readers the trouble of figuring out the rationale all over again when considering future changes to this code.
quoted hunk ↗ jump to hunk
--- a/git-send-email.perl +++ b/git-send-email.perl@@ -1313,7 +1313,7 @@ foreach my $t (@files) { # set up for the next message if ($thread && $message_was_sent && - (chain_reply_to() || !defined $reply_to || length($reply_to) == 0)) { + ($message_num == 1 || chain_reply_to() || !defined $reply_to || length($reply_to) == 0)) { $reply_to = $message_id;
Would it be possible to break this long line? If you're feeling particularly adventurous, it would be nice to add a test for the changed functionality to t/t9001-send-email.sh, so we don't break it with other changes in the future. I haven't looked too deeply or even tried running applying the patch, but generally it looks good to me. Ciao, Jonathan