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@xxxxxxx/T/#u > + replies: Brian M Carlson, Taylor Blau > v1 - https://public-inbox.org/git/20221111021502.449662-1-michael.strawbridge@xxxxxxx/T/#t > + replies: Luben Tuikov, Ævar Arnfjörð Bjarmason > v2 - https://public-inbox.org/git/20221111193042.641898-1-michael.strawbridge@xxxxxxx/T/#t > v3 - https://public-inbox.org/git/20221111194223.644845-1-michael.strawbridge@xxxxxxx/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
