[MERGED] libosmo-sccp[master]: contrib: Enable -Werror by default

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

Harald Welte gerrit-no-reply at lists.osmocom.org
Sat Jul 8 10:46:32 UTC 2017


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 <pespin at sysmocom.de>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder



More information about the gerrit-log mailing list