osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-msc/+/30155 )
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 OsmoMSC with --enable-werror too.
Related: OS#4462 Change-Id: Id54be9dd1aa66cc27eb5ee4010be9e495865b331 --- M contrib/jenkins.sh 1 file changed, 2 insertions(+), 5 deletions(-)
Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved
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