Re: ms/sendemail-validate-headers, was Re: What's cooking in git.git (Nov 2022, #03; Mon, 14)

From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: Nov 18, 2022, 13:24 UTC
Message-ID: <40q20nq9-4qsq-8130-3783-8o97o06824qq@tzk.qr>
In-Reply-To: <874juzglx8.fsf@amd.com>

Hi Michael,

On Wed, 16 Nov 2022, Strawbridge, Michael wrote:

> For ms/sendemail-validate-headers (2022-11-11) 1 commit:
>
> My apologies.  I think not having the patches chained in the same email
> is causing confusion.

Please do not apologize for something that is not at all your fault. It is an unstructured mailing list, after all.

Show 16 quoted lines

> I did have some review on these emails:
>
> v0 - https://public-inbox.org/git/20221109182254.71967-1-michael.strawbridge@amd.com/T/#u
>   + replies: Brian M Carlson, Taylor Blau
> v1 - https://public-inbox.org/git/20221111021502.449662-1-michael.strawbridge@amd.com/T/#t
>   + replies: Luben Tuikov, Ævar Arnfjörð Bjarmason
> v2 - https://public-inbox.org/git/20221111193042.641898-1-michael.strawbridge@amd.com/T/#t
> v3 - https://public-inbox.org/git/20221111194223.644845-1-michael.strawbridge@amd.com/T/#t
>   + replies: Johannes Schindelin
>
> I'm currently in the process of trying to run the T9001 tests.  I am
> getting: error: test_bool_env requires bool values both for
> $GIT_TEST_PASSING_SANITIZE_LEAK and for the default fallback.  I do
> wonder if the change in validation location (as Brian mentioned) is
> changing output (even though the new location is needed to have all
> information for headers).  I'll keep looking into it.

Thank you! Dscho