Re: [PATCH] test-lib: unset trace2 parent envvars
On 2022.01.18 13:50, Junio C Hamano wrote:
Show 13 quoted lines
> Josh Steadmon <steadmon@google.com> writes:
>
> > Yes, due to the number of different TRACE2 modes, I figured it was
> > cleaner to allow them by default and then explicitly unset the ones that
> > cause trouble for trace parsing tests.
>
> I think I would agree that it is a better solution, and the design
> behind it (which makes me say it is better) should be recorded in
> the log message (and possibly in-code comment near the place we
> reset them) to help future developers who may add more TRACE2
> variables.
>
> Thanks.