Skip to content

Commit cd5c281

Browse files
navytuxgitster
authored andcommitted
t/perf/run: copy config.mak.autogen & friends to build area
Otherwise for people who use autotools-based configure in main worktree, the performance testing results will be inconsistent as work and build trees could be using e.g. different optimization levels. See e.g. http://public-inbox.org/git/20160818175222.bmm3ivjheokf2qzl@sigill.intra.peff.net/ for example. NOTE config.status has to be copied because otherwise without it the build would want to run reconfigure this way loosing just copied config.mak.autogen. Signed-off-by: Kirill Smelkov <kirr@nexedi.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
1 parent 49981d8 commit cd5c281

File tree

1 file changed

+7
-1
lines changed

1 file changed

+7
-1
lines changed

t/perf/run

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,13 @@ unpack_git_rev () {
3030
}
3131
build_git_rev () {
3232
rev=$1
33-
cp ../../config.mak build/$rev/config.mak
33+
for config in config.mak config.mak.autogen config.status
34+
do
35+
if test -e "../../$config"
36+
then
37+
cp "../../$config" "build/$rev/"
38+
fi
39+
done
3440
(cd build/$rev && make $GIT_PERF_MAKE_OPTS) ||
3541
die "failed to build revision '$mydir'"
3642
}

0 commit comments

Comments
 (0)