d_
is regular file. Some people might have config.mak as a symlink for
example. We don't want to miss them too.
Please find updated patch below:
8<
From: Kirill Smelkov
Subject: [PATCH] t/perf/run: Don't forget to copy config.mak.autogen & friends
to build area
Other
Junio C Hamano writes:
> In other words, something along this line, perhaps.
> ...
Not quite. There is no guanratee that the user is using autoconf at
all. It should be more like this, I think.
t/perf/run | 8 +++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/t/perf/run b
Junio C Hamano writes:
>> build_git_rev () {
>> rev=$1
>> -cp ../../config.mak build/$rev/config.mak
>> +cp -t build/$rev ../../{config.mak,config.mak.autogen,config.status}
>
> That unfortunately is a GNUism -t with a bash-ism {a,b,c}; just keep
> it simple and stupid to make sure
Kirill Smelkov writes:
> 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/2016081817522
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.bmm3ivjheokf2...@sigill.intra.peff.
5 matches
Mail list logo