Hello Neels Hofmeyr, Jenkins Builder,

I'd like you to reexamine a change.  Please visit

    https://gerrit.osmocom.org/2954

to look at the new patch set (#8).

contrib: Enable -Werror by default

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


  git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/54/2954/8

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 0c11682..820ac45 100755
--- a/contrib/jenkins.sh
+++ b/contrib/jenkins.sh
@@ -5,7 +5,7 @@
 ./contrib/verify_value_string_arrays_are_terminated.py $(find . -name "*.[hc]")
 
 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 \
@@ -17,7 +17,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: newpatchset
Gerrit-Change-Id: I1752cad1e34cb6e5f36edf3f538d803eff29d18c
Gerrit-PatchSet: 8
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Owner: Pau Espin Pedrol <pes...@sysmocom.de>
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>

Reply via email to