Junio C Hamano wrote:
> Christian Couder <chriscool@xxxxxxxxxxxxx> writes:
> > Signed-off-by: Christian Couder <chriscool@xxxxxxxxxxxxx>
> > ---
> > trace.c | 3 ++-
> > 1 files changed, 2 insertions(+), 1 deletions(-)
> >
> > diff --git a/trace.c b/trace.c
> > index f9efc91..495e5ed 100644
> > --- a/trace.c
> > +++ b/trace.c
> > @@ -55,7 +55,8 @@ static int get_trace_fd(int *need_close)
> > {
> > char *trace = getenv("GIT_TRACE");
> >
> > - if (!trace || !strcmp(trace, "0") || !strcasecmp(trace, "false"))
> > + if (!trace || !strcmp(trace, "") ||
> > + !strcmp(trace, "0") || !strcasecmp(trace, "false"))
> > return 0;
> > if (!strcmp(trace, "1") || !strcasecmp(trace, "true"))
> > return STDERR_FILENO;
> > --
> > 1.4.3.rc2.gadba
>
> This means that
>
> GIT_TRACE= git-foo
> and
> unset GIT_TRACE ; git-foo
>
> should behave the same way, right?
Yes, that's the goal of this patch.
> If so that sounds sensible.
Thanks,
Christian.
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html