Michael Strawbridge <michael.strawbridge@xxxxxxx> writes:
> Subject: [PATCH] send-email: move validation code below process_address_list
>
> Move validation logic below processing of email address lists so that
> email validation gets the proper email addresses.
Hmph, without this patch, the tip of 'seen' passes t9001 on my box,
but with it, it claims that it failed 58, 87, and 89.
Here is how #58 fails (the last part of "cd t && sh t9001-*.sh -i -v").
expecting success of 9001.58 'In-Reply-To without --chain-reply-to':
clean_fake_sendmail &&
echo "<unique-message-id@xxxxxxxxxxx>" >expect &&
git send-email \
--from="Example <nobody@xxxxxxxxxxx>" \
--to=nobody@xxxxxxxxxxx \
--no-chain-reply-to \
--in-reply-to="$(cat expect)" \
--smtp-server="$(pwd)/fake.sendmail" \
$patches $patches $patches \
2>errors &&
# The first message is a reply to --in-reply-to
sed -n -e "s/^In-Reply-To: *\(.*\)/\1/p" msgtxt1 >actual &&
test_cmp expect actual &&
# Second and subsequent messages are replies to the first one
sed -n -e "s/^Message-ID: *\(.*\)/\1/p" msgtxt1 >expect &&
sed -n -e "s/^In-Reply-To: *\(.*\)/\1/p" msgtxt2 >actual &&
test_cmp expect actual &&
sed -n -e "s/^In-Reply-To: *\(.*\)/\1/p" msgtxt3 >actual &&
test_cmp expect actual
0001-Second.patch
0001-Second.patch
0001-Second.patch
(mbox) Adding cc: A <author@xxxxxxxxxxx> from line 'From: A <author@xxxxxxxxxxx>'
(mbox) Adding cc: One <one@xxxxxxxxxxx> from line 'Cc: One <one@xxxxxxxxxxx>, two@xxxxxxxxxxx'
(mbox) Adding cc: two@xxxxxxxxxxx from line 'Cc: One <one@xxxxxxxxxxx>, two@xxxxxxxxxxx'
(body) Adding cc: C O Mitter <committer@xxxxxxxxxxx> from line 'Signed-off-by: C O Mitter <committer@xxxxxxxxxxx>'
OK. Log says:
Sendmail: /usr/local/google/home/jch/w/git.git/t/trash directory.t9001-send-email/fake.sendmail -i nobody@xxxxxxxxxxx author@xxxxxxxxxxx one@xxxxxxxxxxx two@xxxxxxxxxxx committer@xxxxxxxxxxx
From: Example <nobody@xxxxxxxxxxx>
To: nobody@xxxxxxxxxxx
Cc: A <author@xxxxxxxxxxx>,
One <one@xxxxxxxxxxx>,
two@xxxxxxxxxxx,
C O Mitter <committer@xxxxxxxxxxx>
Subject: [PATCH 1/1] Second.
Date: Tue, 24 Oct 2023 21:52:27 +0000
Message-ID: <20231024215229.1787922-1-nobody@xxxxxxxxxxx>
X-Mailer: git-send-email 2.42.0-705-g1a1f985ecc
In-Reply-To: <unique-message-id@xxxxxxxxxxx>
References: <unique-message-id@xxxxxxxxxxx>
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
Result: OK
(mbox) Adding cc: A <author@xxxxxxxxxxx> from line 'From: A <author@xxxxxxxxxxx>'
(mbox) Adding cc: One <one@xxxxxxxxxxx> from line 'Cc: One <one@xxxxxxxxxxx>, two@xxxxxxxxxxx'
(mbox) Adding cc: two@xxxxxxxxxxx from line 'Cc: One <one@xxxxxxxxxxx>, two@xxxxxxxxxxx'
(body) Adding cc: C O Mitter <committer@xxxxxxxxxxx> from line 'Signed-off-by: C O Mitter <committer@xxxxxxxxxxx>'
OK. Log says:
Sendmail: /usr/local/google/home/jch/w/git.git/t/trash directory.t9001-send-email/fake.sendmail -i nobody@xxxxxxxxxxx author@xxxxxxxxxxx one@xxxxxxxxxxx two@xxxxxxxxxxx committer@xxxxxxxxxxx
From: Example <nobody@xxxxxxxxxxx>
To: nobody@xxxxxxxxxxx
Cc: A <author@xxxxxxxxxxx>,
One <one@xxxxxxxxxxx>,
two@xxxxxxxxxxx,
C O Mitter <committer@xxxxxxxxxxx>
Subject: [PATCH 1/1] Second.
Date: Tue, 24 Oct 2023 21:52:28 +0000
Message-ID: <20231024215229.1787922-2-nobody@xxxxxxxxxxx>
X-Mailer: git-send-email 2.42.0-705-g1a1f985ecc
In-Reply-To: <unique-message-id@xxxxxxxxxxx>
References: <unique-message-id@xxxxxxxxxxx>
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
Result: OK
(mbox) Adding cc: A <author@xxxxxxxxxxx> from line 'From: A <author@xxxxxxxxxxx>'
(mbox) Adding cc: One <one@xxxxxxxxxxx> from line 'Cc: One <one@xxxxxxxxxxx>, two@xxxxxxxxxxx'
(mbox) Adding cc: two@xxxxxxxxxxx from line 'Cc: One <one@xxxxxxxxxxx>, two@xxxxxxxxxxx'
(body) Adding cc: C O Mitter <committer@xxxxxxxxxxx> from line 'Signed-off-by: C O Mitter <committer@xxxxxxxxxxx>'
OK. Log says:
Sendmail: /usr/local/google/home/jch/w/git.git/t/trash directory.t9001-send-email/fake.sendmail -i nobody@xxxxxxxxxxx author@xxxxxxxxxxx one@xxxxxxxxxxx two@xxxxxxxxxxx committer@xxxxxxxxxxx
From: Example <nobody@xxxxxxxxxxx>
To: nobody@xxxxxxxxxxx
Cc: A <author@xxxxxxxxxxx>,
One <one@xxxxxxxxxxx>,
two@xxxxxxxxxxx,
C O Mitter <committer@xxxxxxxxxxx>
Subject: [PATCH 1/1] Second.
Date: Tue, 24 Oct 2023 21:52:29 +0000
Message-ID: <20231024215229.1787922-3-nobody@xxxxxxxxxxx>
X-Mailer: git-send-email 2.42.0-705-g1a1f985ecc
In-Reply-To: <unique-message-id@xxxxxxxxxxx>
References: <unique-message-id@xxxxxxxxxxx>
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
Result: OK
--- expect 2023-10-24 21:52:29.115044899 +0000
+++ actual 2023-10-24 21:52:29.119045306 +0000
@@ -1 +1 @@
-<20231024215229.1787922-1-nobody@xxxxxxxxxxx>
+<unique-message-id@xxxxxxxxxxx>
not ok 58 - In-Reply-To without --chain-reply-to
#
# clean_fake_sendmail &&
# echo "<unique-message-id@xxxxxxxxxxx>" >expect &&
# git send-email \
# --from="Example <nobody@xxxxxxxxxxx>" \
# --to=nobody@xxxxxxxxxxx \
# --no-chain-reply-to \
# --in-reply-to="$(cat expect)" \
# --smtp-server="$(pwd)/fake.sendmail" \
# $patches $patches $patches \
# 2>errors &&
# # The first message is a reply to --in-reply-to
# sed -n -e "s/^In-Reply-To: *\(.*\)/\1/p" msgtxt1 >actual &&
# test_cmp expect actual &&
# # Second and subsequent messages are replies to the first one
# sed -n -e "s/^Message-ID: *\(.*\)/\1/p" msgtxt1 >expect &&
# sed -n -e "s/^In-Reply-To: *\(.*\)/\1/p" msgtxt2 >actual &&
# test_cmp expect actual &&
# sed -n -e "s/^In-Reply-To: *\(.*\)/\1/p" msgtxt3 >actual &&
# test_cmp expect actual
#
1..58
