Thread (3 messages) 3 messages, 2 authors, 2023-10-25

Re: [PATCH] send-email: move validation code below process_address_list

From: Michael Strawbridge <hidden>
Date: 2023-10-25 18:48:50

Possibly related (same subject, not in this thread)


On 10/24/23 18:03, Junio C Hamano wrote:
Junio C Hamano [off-list ref] writes:
quoted
Michael Strawbridge [off-list ref] writes:
quoted
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.
FWIW, when this patch is used with 'master' (not 'seen'), t9001
claims the same three tests failed.  The way #58 fails seems to be
identical to the way 'seen' with this patch failed, shown in the
message I am responding to.
I'm sorry to have wasted your time with patch 1. I had done the other manual
tests but ended up forgetting the automated ones.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help