Re: [PATCH 1/2] trace2: rename environment variables to GIT_TRACE2*
- From
Junio C Hamano <gitster@pobox.com>
- Date
- May 28, 2019, 16:25 UTC
- Message-ID
- <xmqq36ky5ywr.fsf@gitster-ct.c.googlers.com>
- In-Reply-To
- <20190519144309.9597-1-szeder.dev@gmail.com>
SZEDER Gábor <szeder.dev@gmail.com> writes:
> For an environment variable that is supposed to be set by users, the > GIT_TR2* env vars are just too unclear, inconsistent, and ugly.
FWIW, I personally am in favor of this change and would prefer to see this done before the use of the names with unguessable abbreviation gets ingrained too deeply.
I do not see any objections around these two patches after waiting for a week or so, but I do not see any Yay!s, either, so ...
Show 5 quoted lines
> OTOH, we don't gain anything at all by omitting the last three > characters of "trace" from the names of these environment variables. > > So let's rename all GIT_TR2* environment variables to GIT_TRACE2*, > before they make their way into a stable release.