Harald Welte has submitted this change and it was merged. ( https://gerrit.osmocom.org/9536 )
Change subject: jenkins_*.sh: add --enable-werror to configure flags ...................................................................... jenkins_*.sh: add --enable-werror to configure flags The flag is enabled for all BTS flavour except for lc15, which still contain several compilation warnings (fixes submitted to gerrit, but not merged since no contributor is testing lc15 nowadays). Change-Id: Ib65056633697dafa63831fc2a480b798df550db6 --- M contrib/jenkins_bts_trx.sh M contrib/jenkins_lc15.sh M contrib/jenkins_oct.sh M contrib/jenkins_oct_and_bts_trx.sh M contrib/jenkins_sysmobts.sh 5 files changed, 20 insertions(+), 4 deletions(-) Approvals: Jenkins Builder: Verified Harald Welte: Looks good to me, approved Vadim Yanitskiy: Looks good to me, but someone else must approve diff --git a/contrib/jenkins_bts_trx.sh b/contrib/jenkins_bts_trx.sh index 3e3d8d3..1b723e6 100755 --- a/contrib/jenkins_bts_trx.sh +++ b/contrib/jenkins_bts_trx.sh @@ -14,9 +14,10 @@ cd "$deps" configure_flags="\ + --enable-sanitize \ + --enable-werror \ --with-osmo-pcu=$deps/osmo-pcu/include \ --enable-trx \ - --enable-sanitize \ " build_bts "osmo-bts-trx" "$configure_flags" diff --git a/contrib/jenkins_lc15.sh b/contrib/jenkins_lc15.sh index 38c1f33..c7d62c9 100755 --- a/contrib/jenkins_lc15.sh +++ b/contrib/jenkins_lc15.sh @@ -14,7 +14,11 @@ cd "$deps" osmo-layer1-headers.sh lc15 "$FIRMWARE_VERSION" -configure_flags="--enable-sanitize --with-litecell15=$deps/layer1-headers/inc/ --enable-litecell15" +configure_flags="\ + --enable-sanitize \ + --with-litecell15=$deps/layer1-headers/inc/ \ + --enable-litecell15 \ + " build_bts "osmo-bts-lc15" "$configure_flags" diff --git a/contrib/jenkins_oct.sh b/contrib/jenkins_oct.sh index efbd368..bd57dd1 100755 --- a/contrib/jenkins_oct.sh +++ b/contrib/jenkins_oct.sh @@ -14,7 +14,12 @@ cd "$deps" osmo-layer1-headers.sh oct "$FIRMWARE_VERSION" -configure_flags="--enable-sanitize --with-octsdr-2g=$deps/layer1-headers/ --enable-octphy" +configure_flags="\ + --enable-sanitize \ + --enable-werror \ + --with-octsdr-2g=$deps/layer1-headers/ \ + --enable-octphy \ + " build_bts "osmo-bts-octphy" "$configure_flags" diff --git a/contrib/jenkins_oct_and_bts_trx.sh b/contrib/jenkins_oct_and_bts_trx.sh index f68a9d3..049f5f7 100755 --- a/contrib/jenkins_oct_and_bts_trx.sh +++ b/contrib/jenkins_oct_and_bts_trx.sh @@ -16,6 +16,7 @@ osmo-layer1-headers.sh oct "$FIRMWARE_VERSION" configure_flags="\ + --enable-werror \ --with-osmo-pcu=$deps/osmo-pcu/include \ --with-octsdr-2g=$deps/layer1-headers/ \ --enable-octphy \ diff --git a/contrib/jenkins_sysmobts.sh b/contrib/jenkins_sysmobts.sh index 7488419..d0d05ae 100755 --- a/contrib/jenkins_sysmobts.sh +++ b/contrib/jenkins_sysmobts.sh @@ -16,7 +16,12 @@ mkdir -p "$inst/include/sysmocom/femtobts" ln -s $deps/layer1-headers/include/* "$inst/include/sysmocom/femtobts/" -configure_flags="--enable-sanitize --enable-sysmocom-bts --with-sysmobts=$inst/include/" +configure_flags="\ + --enable-sanitize \ + --enable-werror \ + --enable-sysmocom-bts \ + --with-sysmobts=$inst/include/ \ + " # This will not work for the femtobts if [ $FIRMWARE_VERSION != "femtobts_v2.7" ]; then -- To view, visit https://gerrit.osmocom.org/9536 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-bts Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ib65056633697dafa63831fc2a480b798df550db6 Gerrit-Change-Number: 9536 Gerrit-PatchSet: 4 Gerrit-Owner: Pau Espin Pedrol <[email protected]> Gerrit-Reviewer: Harald Welte <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: Vadim Yanitskiy <[email protected]>
