This is an automated email from the ASF dual-hosted git repository.
bcall 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 86106dc Fix and Update build ordering of lib
86106dc is described below
commit 86106dcd68f878ad0b1c059002354877c58cc45e
Author: Xavier Chi <[email protected]>
AuthorDate: Thu Jun 21 13:02:39 2018 -0500
Fix and Update build ordering of lib
---
ci/jenkins/bin/clang-analyzer.sh | 6 ++++++
lib/Makefile.am | 3 ++-
2 files changed, 8 insertions(+), 1 deletion(-)
diff --git a/ci/jenkins/bin/clang-analyzer.sh b/ci/jenkins/bin/clang-analyzer.sh
index 471d31f..d3b5b87 100755
--- a/ci/jenkins/bin/clang-analyzer.sh
+++ b/ci/jenkins/bin/clang-analyzer.sh
@@ -75,6 +75,12 @@ ${LLVM_BASE}/bin/scan-build ./configure ${configure} \
CXXFLAGS="-stdlib=libc++ -I${LLVM_BASE}/include/c++/v1 -std=c++17" \
LDFLAGS="-L${LLVM_BASE}/lib64 -Wl,-rpath=${LLVM_BASE}/lib64"
+# Since we don't want the analyzer to look at yamlcpp, build it first
+# without scan-build. The subsequent make will then skip it.
+# the all-local can be taken out and lib changed to lib/yamlcpp
+# by making yaml cpp a SUBDIRS in lib/Makefile.am.
+${ATS_MAKE} -j $NPROCS -C lib all-local V=1 Q=
+
${LLVM_BASE}/bin/scan-build ${checkers} ${options} -o ${output} \
--html-title="clang-analyzer: ${ATS_BRANCH}${github_pr}" \
${ATS_MAKE} -j $NPROCS V=1 Q=
diff --git a/lib/Makefile.am b/lib/Makefile.am
index c07ab92..42f187a 100644
--- a/lib/Makefile.am
+++ b/lib/Makefile.am
@@ -16,7 +16,7 @@
# See the License for the specific language governing permissions and
# limitations under the License.
-SUBDIRS = ts records . tsconfig cppapi
+SUBDIRS = . ts records tsconfig cppapi
if BUILD_PERL_LIB
SUBDIRS += perl
@@ -26,6 +26,7 @@ if BUILD_WCCP
SUBDIRS += wccp
endif
+# to prevent Clang Analyzer warning
LOCAL =
if BUILD_YAML_CPP
