[PATCH] t/perf/run: Don't forget to copy config.mak.autogen & friends to build area
- From
Kirill Smelkov <kirr@nexedi.com>
- Date
- Sep 10, 2016, 15:05 UTC
- Message-ID
- <20160910150512.19473-1-kirr@nexedi.com>
- In-Reply-To
- <20160910145738.x6ihp2gqzpk7dbi3@teco.navytux.spb.ru>
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> --- ( Resending as separate patch-mail, just in case )
t/perf/run | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/t/perf/run b/t/perf/run index cfd7012..aa383c2 100755 --- a/t/perf/run +++ b/t/perf/run @@ -30,7 +30,7 @@ unpack_git_rev () { } build_git_rev () { rev=$1 - cp ../../config.mak build/$rev/config.mak + cp -t build/$rev ../../{config.mak,config.mak.autogen,config.status} (cd build/$rev && make $GIT_PERF_MAKE_OPTS) || die "failed to build revision '$mydir'" }
-- 2.9.2.701.gf965a18.dirty