Harald Welte has submitted this change and it was merged.

Change subject: contrib: Enable -Werror by default
......................................................................


contrib: Enable -Werror by default

Change-Id: I16a8ca7f0bfabeed6a9b58cda5e60bd58889f74d
---
M configure.ac
M contrib/jenkins.sh
2 files changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Harald Welte: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/configure.ac b/configure.ac
index 200f55d..7e3e03e 100644
--- a/configure.ac
+++ b/configure.ac
@@ -54,7 +54,7 @@
 AC_SUBST(SYMBOL_VISIBILITY)
 
 CFLAGS="$CFLAGS -Wall"
-CPPFLAGS="$CFLAGS -Wall"
+CPPFLAGS="$CPPFLAGS -Wall"
 
 AC_OUTPUT(
     libosmo-sigtran.pc
diff --git a/contrib/jenkins.sh b/contrib/jenkins.sh
index 668a58b..897c464 100755
--- a/contrib/jenkins.sh
+++ b/contrib/jenkins.sh
@@ -29,7 +29,7 @@
 set -x
 
 autoreconf --install --force
-./configure
+./configure CFLAGS="-Werror" CPPFLAGS="-Werror"
 $MAKE $PARALLEL_MAKE
 $MAKE distcheck \
   || cat-testlogs.sh

-- 
To view, visit https://gerrit.osmocom.org/3136
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I16a8ca7f0bfabeed6a9b58cda5e60bd58889f74d
Gerrit-PatchSet: 2
Gerrit-Project: libosmo-sccp
Gerrit-Branch: master
Gerrit-Owner: Pau Espin Pedrol <pes...@sysmocom.de>
Gerrit-Reviewer: Harald Welte <lafo...@gnumonks.org>
Gerrit-Reviewer: Jenkins Builder

Reply via email to