undef/git-commit-vandalism
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Makefile: make perf tests optional for profile build

Browse Source 
The perf tests need a repository to operate on; if none is
defined, we fall back to the repository containing our build
directory.  That fails, though, for an exported tarball of
git.git, which has no repository.

Since 5d7fd6d we run the perf tests as part of "make
profile". Therefore "make profile" fails out of the box on
released tarballs of v2.1.0.

We can fix this by making the perf tests optional; if they
are skipped, we still run the regular test suite, which
should give a lot of profile data (and is what we used to do
prior to 5d7fd6d anyway).

Signed-off-by: Jeff King <peff@peff.net>
Acked-by: Andi Kleen <ak@linux.intel.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:
Jeff King 2014-08-19 02:12:03 -04:00 committed by Junio C Hamano
parent 066dd2632a
commit 93b5393611
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
Makefile
View File

@ -1649,7 +1649,11 @@ endif
profile:: profile-clean profile:: profile-clean
$(MAKE) PROFILE=GEN all $(MAKE) PROFILE=GEN all
$(MAKE) PROFILE=GEN -j1 test $(MAKE) PROFILE=GEN -j1 test
$(MAKE) PROFILE=GEN -j1 perf @if test -n "$$GIT_PERF_REPO" || test -d .git; then \
$(MAKE) PROFILE=GEN -j1 perf; \
else \
echo "Skipping profile of perf tests..."; \
fi
$(MAKE) PROFILE=USE all $(MAKE) PROFILE=USE all
profile-fast: profile-clean profile-fast: profile-clean