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

[Johannes Schindelin <Johannes.Schindelin@gmx.de>]

[-- Attachment #1: Type: text/plain, Size: 1340 bytes --]

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.

> 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