summaryrefslogtreecommitdiff
path: root/t/perf/run
diff options
context:
space:
mode:
authorÆvar Arnfjörð Bjarmason <avarab@gmail.com>2019-05-07 10:54:34 (GMT)
committerJunio C Hamano <gitster@pobox.com>2019-05-08 02:00:28 (GMT)
commit82b7eb231d146678317fd4cf63a646c4233976e0 (patch)
tree449c83c597c12beaa2e671c12679766c782dffbf /t/perf/run
parentfab80ee79ddf59a5d00812005bef0fa3acf5b6bf (diff)
downloadgit-82b7eb231d146678317fd4cf63a646c4233976e0.zip
git-82b7eb231d146678317fd4cf63a646c4233976e0.tar.gz
git-82b7eb231d146678317fd4cf63a646c4233976e0.tar.bz2
perf-lib.sh: forbid the use of GIT_TEST_INSTALLED
As noted in preceding commits setting GIT_TEST_INSTALLED has never been supported or documented, and as noted in an earlier t/perf/README change to the extent that it's been documented nobody's notices that the example hasn't worked since 3c8f12c96c ("test-lib: reorder and include GIT-BUILD-OPTIONS a lot earlier", 2012-06-24). We could directly support GIT_TEST_INSTALLED for invocations without the "run" script, such as: GIT_TEST_INSTALLED=../../ ./p0000-perf-lib-sanity.sh GIT_TEST_INSTALLED=/home/avar/g/git ./p0000-perf-lib-sanity.sh But while not having this "error" will "work", it won't write the the resulting "test-results/*" files to the right place, and thus a subsequent call to aggregate.perl won't work as expected. Let's just tell the user that they need to use the "run" script, which'll correctly deal with this and set the right PERF_RESULTS_PREFIX. If someone's in desperate need of bypassing "run" for whatever reason they can trivially do so by setting "PERF_SET_GIT_TEST_INSTALLED", but not we won't have people who expect GIT_TEST_INSTALLED to just work wondering why their aggregation doesn't work, even though they're running the right "git". Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Diffstat (limited to 't/perf/run')
-rwxr-xr-xt/perf/run2
1 files changed, 2 insertions, 0 deletions
diff --git a/t/perf/run b/t/perf/run
index cd3882b..c7b8610 100755
--- a/t/perf/run
+++ b/t/perf/run
@@ -84,6 +84,8 @@ set_git_test_installed () {
GIT_TEST_INSTALLED=$mydir_abs
fi
export GIT_TEST_INSTALLED
+ PERF_SET_GIT_TEST_INSTALLED=true
+ export PERF_SET_GIT_TEST_INSTALLED
}
run_dirs_helper () {