osmith has submitted this change. (
https://gerrit.osmocom.org/c/osmo-sgsn/+/30156 )
Change subject: contrib/jenkins.sh: use enable-werror with IU too
......................................................................
contrib/jenkins.sh: use enable-werror with IU too
Now that the warnings in osmo-iuh have been fixed, we should be able to
build the IU version of OsmoSGSN with --enable-werror too.
Related: OS#4462
Change-Id: I8cc4e209e21acfe513bef72927499f1ccdead783
---
M contrib/jenkins.sh
1 file changed, 2 insertions(+), 5 deletions(-)
Approvals:
Jenkins Builder: Verified
laforge: Looks good to me, approved
pespin: Looks good to me, but someone else must approve
diff --git a/contrib/jenkins.sh b/contrib/jenkins.sh
index 2325897..321beef 100755
--- a/contrib/jenkins.sh
+++ b/contrib/jenkins.sh
@@ -37,14 +37,11 @@
osmo-build-dep.sh osmo-ggsn
osmo-build-dep.sh osmo-hlr
-enable_werror=""
if [ "x$IU" = "x--enable-iu" ]; then
osmo-build-dep.sh libosmo-sccp
osmo-build-dep.sh libasn1c
#osmo-build-dep.sh asn1c aper-prefix # only needed for make regen in
osmo-iuh
osmo-build-dep.sh osmo-iuh
-else
- enable_werror="--enable-werror"
fi
# Additional configure options and depends
@@ -63,12 +60,12 @@
cd "$base"
autoreconf --install --force
-./configure --enable-sanitize $enable_werror $IU --enable-external-tests
$CONFIG
+./configure --enable-sanitize --enable-werror $IU --enable-external-tests
$CONFIG
$MAKE $PARALLEL_MAKE
LD_LIBRARY_PATH="$inst/lib" $MAKE check \
|| cat-testlogs.sh
LD_LIBRARY_PATH="$inst/lib" \
- DISTCHECK_CONFIGURE_FLAGS="$enable_werror $IU --enable-external-tests
$CONFIG" \
+ DISTCHECK_CONFIGURE_FLAGS="--enable-werror $IU --enable-external-tests
$CONFIG" \
$MAKE $PARALLEL_MAKE distcheck \
|| cat-testlogs.sh
--
To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/30156
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: I8cc4e209e21acfe513bef72927499f1ccdead783
Gerrit-Change-Number: 30156
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: osmith <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-MessageType: merged
