You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by so...@apache.org on 2017/07/27 20:31:33 UTC
[trafficserver] branch 6.2.x updated (86102d2 -> fa6c3bc)
This is an automated email from the ASF dual-hosted git repository.
sorber pushed a change to branch 6.2.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git.
from 86102d2 TS-4930: Removes an old test file, it duplicates HdrTest.cc
new 749737b Updated with some clang-analyzer tweaks done for the CI
new ba704e6 TS-4566 Disabled clang-analyzer on LuaJIT
new 7447a57 TS-4566: Try harder to exclude LuaJIT from static analysis.
new fa6c3bc Avoid varargs warnings when compiling LuaJIT
The 4 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "add" were already present in the repository and have only
been added to this reference.
Summary of changes:
ci/jenkins/bin/clang-analyzer.sh | 28 +++++++++++++++++++++++-----
ci/jenkins/bin/environment.sh | 2 +-
configure.ac | 6 +++---
3 files changed, 27 insertions(+), 9 deletions(-)
--
To stop receiving notification emails like this one, please contact
['"commits@trafficserver.apache.org" <co...@trafficserver.apache.org>'].
[trafficserver] 04/04: Avoid varargs warnings when compiling LuaJIT
Posted by so...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
sorber pushed a commit to branch 6.2.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git
commit fa6c3bc9d4228013d89c8b071cb89d80f3bb8e7a
Author: Leif Hedstrom <zw...@apache.org>
AuthorDate: Tue Apr 4 12:22:30 2017 -0600
Avoid varargs warnings when compiling LuaJIT
(cherry picked from commit bf3781661003a6da7102d891b6b8e3df65c091e5)
Conflicts:
configure.ac
---
configure.ac | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/configure.ac b/configure.ac
index a52c1c2..ec4b9f4 100644
--- a/configure.ac
+++ b/configure.ac
@@ -809,7 +809,7 @@ case $host_os_def in
debug_opt="-ggdb3 $common_opt -Qunused-arguments"
release_opt="-g $common_opt $optimizing_flags -fno-strict-aliasing -Qunused-arguments"
cxx_opt="-Wno-invalid-offsetof"
- lua_cflags="-Wno-parentheses-equality -Wno-tautological-compare -analyzer-disable-all-checks"
+ lua_cflags="-Wno-parentheses-equality -Wno-tautological-compare -analyzer-disable-all-checks -Wno-varargs"
])
AS_IF([test "x$ax_cv_c_compiler_vendor" = "xgnu"], [
@@ -831,7 +831,7 @@ case $host_os_def in
debug_opt="-g $common_opt"
release_opt="-g $common_opt $optimizing_flags -fno-strict-aliasing"
cxx_opt="-Wno-invalid-offsetof"
- lua_cflags="-Wno-parentheses-equality -Wno-tautological-compare"
+ lua_cflags="-Wno-parentheses-equality -Wno-tautological-compare -Wno-varargs"
], [
AC_MSG_WARN([clang is the only supported compiler on Darwin])
])
@@ -852,7 +852,7 @@ case $host_os_def in
debug_opt="-ggdb3 $common_opt"
release_opt="-g $common_opt $optimizing_flags -fno-strict-aliasing"
cxx_opt="-Wno-invalid-offsetof"
- lua_cflags="-Wno-parentheses-equality -Wno-tautological-compare"
+ lua_cflags="-Wno-parentheses-equality -Wno-tautological-compare -Wno-varargs"
])
AS_IF([test "x$ax_cv_c_compiler_vendor" = "xgnu"], [
--
To stop receiving notification emails like this one, please contact
"commits@trafficserver.apache.org" <co...@trafficserver.apache.org>.
[trafficserver] 01/04: Updated with some clang-analyzer tweaks done
for the CI
Posted by so...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
sorber pushed a commit to branch 6.2.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git
commit 749737b6b493b9745e4979d18b68f816960175dd
Author: Leif Hedstrom <zw...@apache.org>
AuthorDate: Wed May 11 15:32:02 2016 -0700
Updated with some clang-analyzer tweaks done for the CI
(cherry picked from commit 71731c0171cd8feb3b8679ebbd285f0ec1723d51)
---
ci/jenkins/bin/clang-analyzer.sh | 2 +-
ci/jenkins/bin/environment.sh | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/ci/jenkins/bin/clang-analyzer.sh b/ci/jenkins/bin/clang-analyzer.sh
index 8a3dc92..c0dee78 100755
--- a/ci/jenkins/bin/clang-analyzer.sh
+++ b/ci/jenkins/bin/clang-analyzer.sh
@@ -45,7 +45,7 @@ test -d "/home/jenkins/clang-analyzer" && output="/home/jenkins/clang-analyzer"
autoreconf -fi
#scan-build ./configure ${configure}
./configure ${configure}
-scan-build ${checkers} ${options} -o ${output} --html-title="ATS master branch" ${ATS_MAKE} -j4
+/opt/llvm/bin/scan-build ${checkers} ${options} -o ${output} --html-title="ATS master branch" ${ATS_MAKE} -j4
status=$?
${ATS_MAKE} distclean
diff --git a/ci/jenkins/bin/environment.sh b/ci/jenkins/bin/environment.sh
index eabf3e4..0c51679 100755
--- a/ci/jenkins/bin/environment.sh
+++ b/ci/jenkins/bin/environment.sh
@@ -59,7 +59,7 @@ if test "${JOB_NAME#*compiler=clang}" != "${JOB_NAME}"; then
fi
# Figure out parallelism for regular builds / bots
-ATS_MAKE_FLAGS="-j 4"
+ATS_MAKE_FLAGS="-j5"
# This is for Solaris, but didn't seem to work
#[ -x /usr/sbin/psrinfo ] && ATS_MAKE_FLAGS="-j $(/usr/sbin/psrinfo -p)" # Conservative on Solaris
--
To stop receiving notification emails like this one, please contact
"commits@trafficserver.apache.org" <co...@trafficserver.apache.org>.
[trafficserver] 03/04: TS-4566: Try harder to exclude LuaJIT from
static analysis.
Posted by so...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
sorber pushed a commit to branch 6.2.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git
commit 7447a5742ee031c707e3dc7ab117098209007ab9
Author: James Peach <jp...@apache.org>
AuthorDate: Mon Jun 20 16:28:50 2016 -0700
TS-4566: Try harder to exclude LuaJIT from static analysis.
(cherry picked from commit 71b13803b50cf0545f47af0118a9668b1e3a4c2a)
---
ci/jenkins/bin/clang-analyzer.sh | 23 +++++++++++++++++++----
1 file changed, 19 insertions(+), 4 deletions(-)
diff --git a/ci/jenkins/bin/clang-analyzer.sh b/ci/jenkins/bin/clang-analyzer.sh
index 7d5c9fc..454a2cb 100755
--- a/ci/jenkins/bin/clang-analyzer.sh
+++ b/ci/jenkins/bin/clang-analyzer.sh
@@ -23,6 +23,8 @@
# Where are our LLVM tools?
LLVM_BASE=${LLVM:-/opt/llvm}
+NPROCS=${NPROCS:-$(getconf _NPROCESSORS_ONLN)}
+NOCLEAN=${NOCLEAN:-}
# Options
options="--status-bugs --keep-empty"
@@ -45,13 +47,26 @@ test ! -z "${WORKSPACE}" && cd "${WORKSPACE}/src"
output="/tmp"
test -d "/home/jenkins/clang-analyzer" && output="/home/jenkins/clang-analyzer"
+# Tell scan-build to use clang as the underlying compiler to actually build
+# source. If you don't do this, it will default to GCC.
+export CCC_CC=${LLVM_BASE}/bin/clang
+export CCC_CXX=${LLVM_BASE}/bin/clang++
+
autoreconf -fi
-#scan-build ./configure ${configure}
-./configure ${configure} CC=${LLVM_BASE}/bin/clang CXX=${LLVM_BASE}/bin/clang++
-${LLVM_BASE}/bin/scan-build ${checkers} ${options} -o ${output} --html-title="ATS master branch" ${ATS_MAKE} -j4
+scan-build ./configure ${configure}
+
+# Since we don't want the analyzer to look at LuaJIT, build it first
+# without scan-build. The subsequent make will then skip it.
+${ATS_MAKE} -j $NPROCS -C lib all-local V=1 Q=
+
+${LLVM_BASE}/bin/scan-build ${checkers} ${options} -o ${output} --html-title="ATS master branch" ${ATS_MAKE} -j $NPROCS V=1 Q=
status=$?
-${ATS_MAKE} distclean
+# Clean the work area unless NOCLEAN is set. This is jsut for debugging when you
+# need to see what the generated build did.
+if [ ! -z "$NOCLEAN" ]; then
+ ${ATS_MAKE} distclean
+fi
# Cleanup old reports (save the last 10 reports), but only for the CI
if [ "/tmp" != "$output" ]; then
--
To stop receiving notification emails like this one, please contact
"commits@trafficserver.apache.org" <co...@trafficserver.apache.org>.
[trafficserver] 02/04: TS-4566 Disabled clang-analyzer on LuaJIT
Posted by so...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
sorber pushed a commit to branch 6.2.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git
commit ba704e69ad48010ca4749bd25552e41074478eb5
Author: Leif Hedstrom <zw...@apache.org>
AuthorDate: Mon Jun 20 11:25:02 2016 -0600
TS-4566 Disabled clang-analyzer on LuaJIT
(cherry picked from commit 9a3d48485e7652be2a7472fdfa6933ba71a0c139)
---
ci/jenkins/bin/clang-analyzer.sh | 9 ++++++---
configure.ac | 2 +-
2 files changed, 7 insertions(+), 4 deletions(-)
diff --git a/ci/jenkins/bin/clang-analyzer.sh b/ci/jenkins/bin/clang-analyzer.sh
index c0dee78..7d5c9fc 100755
--- a/ci/jenkins/bin/clang-analyzer.sh
+++ b/ci/jenkins/bin/clang-analyzer.sh
@@ -21,9 +21,12 @@
# to talk to the author of it, or ideally, figure out how to get clang-analyzer to
# ignore them ?
+# Where are our LLVM tools?
+LLVM_BASE=${LLVM:-/opt/llvm}
+
# Options
options="--status-bugs --keep-empty"
-configure="--enable-experimental-plugins --enable-cppapi --disable-luajit"
+configure="--enable-experimental-plugins --enable-cppapi"
# Additional checkers
# Phil says these are all FP's: -enable-checker alpha.security.ArrayBoundV2
@@ -44,8 +47,8 @@ test -d "/home/jenkins/clang-analyzer" && output="/home/jenkins/clang-analyzer"
autoreconf -fi
#scan-build ./configure ${configure}
-./configure ${configure}
-/opt/llvm/bin/scan-build ${checkers} ${options} -o ${output} --html-title="ATS master branch" ${ATS_MAKE} -j4
+./configure ${configure} CC=${LLVM_BASE}/bin/clang CXX=${LLVM_BASE}/bin/clang++
+${LLVM_BASE}/bin/scan-build ${checkers} ${options} -o ${output} --html-title="ATS master branch" ${ATS_MAKE} -j4
status=$?
${ATS_MAKE} distclean
diff --git a/configure.ac b/configure.ac
index 5aa9ca8..a52c1c2 100644
--- a/configure.ac
+++ b/configure.ac
@@ -809,7 +809,7 @@ case $host_os_def in
debug_opt="-ggdb3 $common_opt -Qunused-arguments"
release_opt="-g $common_opt $optimizing_flags -fno-strict-aliasing -Qunused-arguments"
cxx_opt="-Wno-invalid-offsetof"
- lua_cflags="-Wno-parentheses-equality -Wno-tautological-compare"
+ lua_cflags="-Wno-parentheses-equality -Wno-tautological-compare -analyzer-disable-all-checks"
])
AS_IF([test "x$ax_cv_c_compiler_vendor" = "xgnu"], [
--
To stop receiving notification emails like this one, please contact
"commits@trafficserver.apache.org" <co...@trafficserver.apache.org>.