osmith submitted this change.

Approvals:
  Jenkins Builder: Verified
  laforge: Looks good to me, approved

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 OsmoMSC with --enable-werror too.

Related: OS#4462
Change-Id: Id54be9dd1aa66cc27eb5ee4010be9e495865b331
---
M contrib/jenkins.sh
1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/contrib/jenkins.sh b/contrib/jenkins.sh
index e1bbad4..558b6dc 100755
--- a/contrib/jenkins.sh
+++ b/contrib/jenkins.sh
@@ -39,13 +39,10 @@
 osmo-build-dep.sh osmo-mgw
 osmo-build-dep.sh osmo-hlr
 
-enable_werror=""
 if [ "x$IU" = "x--enable-iu" ]; then
 	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
@@ -64,12 +61,12 @@
 
 cd "$base"
 autoreconf --install --force
-./configure --enable-sanitize $enable_werror --enable-smpp $IU --enable-external-tests $CONFIG
+./configure --enable-sanitize --enable-werror --enable-smpp $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 --enable-smpp $IU --enable-external-tests $CONFIG" \
+  DISTCHECK_CONFIGURE_FLAGS="--enable-werror --enable-smpp $IU --enable-external-tests $CONFIG" \
   $MAKE $PARALLEL_MAKE distcheck \
   || cat-testlogs.sh

To view, visit change 30155. To unsubscribe, or for help writing mail filters, visit settings.