tests: scrub environment of GIT_* variables
authorJonathan Nieder <jrnieder@gmail.com>
Tue, 15 Mar 2011 10:10:45 +0000 (05:10 -0500)
committerJunio C Hamano <gitster@pobox.com>
Tue, 15 Mar 2011 19:23:57 +0000 (12:23 -0700)
commit95a1d12e9b9faddc02187ca28fdeb4fddd354c59
treeb84c136f4c5d052de663c93b9e6e58c10230d7d4
parent8f323c00dd3c9b396b01a1aeea74f7dfd061bb7f
tests: scrub environment of GIT_* variables

Variables from the inherited environment that are meaningful to git
can break tests in undesirable ways.  For example,

GIT_PAGER=more sh t5400-send-pack.sh -v -i

hangs.  So unset all environment variables in the GIT_ namespace in
test-lib, with a few exceptions:

- GIT_TRACE* are useful for tracking down bugs exhibited by a failing
  test;

- GIT_DEBUG* are GIT_TRACE variables by another name, practically
  speaking.  They should probably be tweaked to follow the
  GIT_TRACE_foo scheme and use trace_printf machinery some time.

- GIT_USE_LOOKUP from v1.5.6-rc0~134^2~1 (sha1-lookup: more memory
  efficient search in sorted list of SHA-1, 2007-12-29) is about
  trying an alternate implementation strategy rather than changing
  semantics and it can be useful to compare performance with and
  without it set.

Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
t/test-lib.sh