You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by zw...@apache.org on 2017/05/25 01:25:37 UTC

[trafficserver] branch master updated: Use proper make wrapper, and parallelize

This is an automated email from the ASF dual-hosted git repository.

zwoop pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git

The following commit(s) were added to refs/heads/master by this push:
       new  d8582f0   Use proper make wrapper, and parallelize
d8582f0 is described below

commit d8582f0610c015ae3a0a8e4823a34ef7f526b7d6
Author: Leif Hedstrom <zw...@apache.org>
AuthorDate: Sun May 21 14:27:09 2017 -0600

    Use proper make wrapper, and parallelize
---
 ci/jenkins/bin/clang-format.sh | 13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)

diff --git a/ci/jenkins/bin/clang-format.sh b/ci/jenkins/bin/clang-format.sh
index 38a5e3f..2d5fbc1 100644
--- a/ci/jenkins/bin/clang-format.sh
+++ b/ci/jenkins/bin/clang-format.sh
@@ -20,8 +20,17 @@ cd "${WORKSPACE}/src"
 autoreconf -if && ./configure
 
 # Test clang-format, copy our version of clang-format if needed
-[ ! -d .git/fmt ] && cp -rp /home/jenkins/fmt .git
-make clang-format
+if [ ! -d .git/fmt ]; then
+    if [ -z "${ghprbTargetBranch}" ]; then
+	cp -rp /home/jenkins/clang-format/master .git/fmt
+    else
+	# This is for Github PR's, to make sure we use the right clang-format for the branch.
+	# This is not an issue on normal branch builds, since they will have the right .git/fmt.
+	cp -rp /home/jenkins/clang-format/${ghprbTargetBranch} .git/fmt
+    fi
+fi
+
+${ATS_MAKE} -j clang-format
 git diff --exit-code
 [ "0" != "$?" ] && exit -1
 

-- 
To stop receiving notification emails like this one, please contact
['"commits@trafficserver.apache.org" <co...@trafficserver.apache.org>'].