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 <[email protected]>
Gerrit-Reviewer: Harald Welte <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Max <[email protected]>
Gerrit-Reviewer: Neels Hofmeyr <[email protected]>
Gerrit-Reviewer: Pau Espin Pedrol <[email protected]>
