Ben Peart <benpeart@xxxxxxxxxxxxx> writes: > diff --git a/t/test-lib.sh b/t/test-lib.sh > index 653688c067..397eb71578 100644 > --- a/t/test-lib.sh > +++ b/t/test-lib.sh > @@ -134,9 +134,9 @@ export EDITOR > GIT_TRACE_BARE=1 > export GIT_TRACE_BARE > > -if test -n "${TEST_GIT_INDEX_VERSION:+isset}" > +if test -n "${GIT_TEST_INDEX_VERSION:+isset}" > then > - GIT_INDEX_VERSION="$TEST_GIT_INDEX_VERSION" > + GIT_INDEX_VERSION="$GIT_TEST_INDEX_VERSION" > export GIT_INDEX_VERSION > fi Is this done a bit before ... > @@ -159,6 +159,7 @@ check_var_migration () { > } > > check_var_migration GIT_FSMONITOR_TEST GIT_TEST_FSMONITOR > +check_var_migration TEST_GIT_INDEX_VERSION GIT_TEST_INDEX_VERSION ... this has a chance to kick in to say things like "Whoa you have TEST_GIT_INDEX_VERSION that is an old spelling of GIT_TEST_INDEX_VERSION", isn't it? > # Add libc MALLOC and MALLOC_PERTURB test > # only if we are not executing the test with valgrind