[MERGED] libosmocore[master]: contrib: Enable -Werror by default

[Pau Espin Pedrol]

Pau Espin Pedrol has submitted this change and it was merged.

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


contrib: Enable -Werror by default

Change-Id: I1752cad1e34cb6e5f36edf3f538d803eff29d18c
---
M contrib/jenkins-arm.sh
M contrib/jenkins.sh
2 files changed, 4 insertions(+), 4 deletions(-)

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



diff --git a/contrib/jenkins-arm.sh b/contrib/jenkins-arm.sh
index d5fc31f..510b0e9 100755
--- a/contrib/jenkins-arm.sh
+++ b/contrib/jenkins-arm.sh
@@ -10,7 +10,7 @@
        --host=arm-none-eabi \
        --enable-embedded \
        --disable-shared \
-       CFLAGS="-Os -ffunction-sections -fdata-sections -nostartfiles 
-nodefaultlibs"
+       CFLAGS="-Os -ffunction-sections -fdata-sections -nostartfiles 
-nodefaultlibs -Werror"
 
 $MAKE $PARALLEL_MAKE \
        || cat-testlogs.sh
@@ -27,7 +27,7 @@
        --host=arm-none-eabi \
        --enable-embedded \
        --disable-shared \
-       CFLAGS="-Os -ffunction-sections -fdata-sections -nostartfiles 
-nodefaultlibs"
+       CFLAGS="-Os -ffunction-sections -fdata-sections -nostartfiles 
-nodefaultlibs -Werror"
 
 $MAKE $PARALLEL_MAKE \
        || cat-testlogs.sh
diff --git a/contrib/jenkins.sh b/contrib/jenkins.sh
index bb6c3cb..6a72840 100755
--- a/contrib/jenkins.sh
+++ b/contrib/jenkins.sh
@@ -11,7 +11,7 @@
 fi
 
 autoreconf --install --force
-./configure --enable-static $ENABLE_SANITIZE
+./configure --enable-static $ENABLE_SANITIZE CFLAGS="-Werror" 
CPPFLAGS="-Werror"
 $MAKE $PARALLEL_MAKE check \
   || cat-testlogs.sh
 $MAKE distcheck \
@@ -23,7 +23,7 @@
 autoreconf --install --force
 mkdir builddir
 cd builddir
-../configure --enable-static
+../configure --enable-static CFLAGS="-Werror" CPPFLAGS="-Werror"
 $MAKE $PARALLEL_MAKE check \
   || cat-testlogs.sh
 $MAKE distcheck \

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I1752cad1e34cb6e5f36edf3f538d803eff29d18c
Gerrit-PatchSet: 13
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Owner: Pau Espin Pedrol <pes...@sysmocom.de>
Gerrit-Reviewer: Harald Welte <lafo...@gnumonks.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Max <msur...@sysmocom.de>
Gerrit-Reviewer: Neels Hofmeyr <nhofm...@sysmocom.de>
Gerrit-Reviewer: Pau Espin Pedrol <pes...@sysmocom.de>