Re: [PATCH] t/perf/run: Don't forget to copy config.mak.autogen & friends to build area

From: Junio C Hamano <gitster@pobox.com>
Date: Sep 12, 2016, 23:10 UTC
Message-ID: <xmqqpoo8yasu.fsf@gitster.mtv.corp.google.com>
In-Reply-To: <xmqqd1k92aif.fsf@gitster.mtv.corp.google.com>

Junio C Hamano <gitster@pobox.com> 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/t/perf/run
index aa383c2..7ec3734 100755
--- a/t/perf/run
+++ b/t/perf/run
@@ -30,7 +30,13 @@ unpack_git_rev () {
 }
 build_git_rev () {
 	rev=$1
-	cp -t build/$rev ../../{config.mak,config.mak.autogen,config.status}
+	for config in config.mak config.mak.autogen config.status
+	do
+		if test -f "../../$config"
+		then
+			cp "../../$config" "build/$rev/"
+		fi
+	done
 	(cd build/$rev && make $GIT_PERF_MAKE_OPTS) ||
 	die "failed to build revision '$mydir'"
 }