<p>Harald Welte <strong>merged</strong> this change.</p><p><a href="https://gerrit.osmocom.org/9536">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  Jenkins Builder: Verified
  Harald Welte: Looks good to me, approved
  Vadim Yanitskiy: Looks good to me, but someone else must approve

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">jenkins_*.sh: add --enable-werror to configure flags<br><br>The flag is enabled for all BTS flavour except for lc15, which still<br>contain several compilation warnings (fixes submitted to gerrit, but not<br>merged since no contributor is testing lc15 nowadays).<br><br>Change-Id: Ib65056633697dafa63831fc2a480b798df550db6<br>---<br>M contrib/jenkins_bts_trx.sh<br>M contrib/jenkins_lc15.sh<br>M contrib/jenkins_oct.sh<br>M contrib/jenkins_oct_and_bts_trx.sh<br>M contrib/jenkins_sysmobts.sh<br>5 files changed, 20 insertions(+), 4 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/contrib/jenkins_bts_trx.sh b/contrib/jenkins_bts_trx.sh</span><br><span>index 3e3d8d3..1b723e6 100755</span><br><span>--- a/contrib/jenkins_bts_trx.sh</span><br><span>+++ b/contrib/jenkins_bts_trx.sh</span><br><span>@@ -14,9 +14,10 @@</span><br><span> cd "$deps"</span><br><span> </span><br><span> configure_flags="\</span><br><span style="color: hsl(120, 100%, 40%);">+  --enable-sanitize \</span><br><span style="color: hsl(120, 100%, 40%);">+  --enable-werror \</span><br><span>   --with-osmo-pcu=$deps/osmo-pcu/include \</span><br><span>   --enable-trx \</span><br><span style="color: hsl(0, 100%, 40%);">-  --enable-sanitize \</span><br><span>   "</span><br><span> </span><br><span> build_bts "osmo-bts-trx" "$configure_flags"</span><br><span>diff --git a/contrib/jenkins_lc15.sh b/contrib/jenkins_lc15.sh</span><br><span>index 38c1f33..c7d62c9 100755</span><br><span>--- a/contrib/jenkins_lc15.sh</span><br><span>+++ b/contrib/jenkins_lc15.sh</span><br><span>@@ -14,7 +14,11 @@</span><br><span> cd "$deps"</span><br><span> osmo-layer1-headers.sh lc15 "$FIRMWARE_VERSION"</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-configure_flags="--enable-sanitize --with-litecell15=$deps/layer1-headers/inc/ --enable-litecell15"</span><br><span style="color: hsl(120, 100%, 40%);">+configure_flags="\</span><br><span style="color: hsl(120, 100%, 40%);">+  --enable-sanitize \</span><br><span style="color: hsl(120, 100%, 40%);">+  --with-litecell15=$deps/layer1-headers/inc/ \</span><br><span style="color: hsl(120, 100%, 40%);">+  --enable-litecell15 \</span><br><span style="color: hsl(120, 100%, 40%);">+  "</span><br><span> </span><br><span> build_bts "osmo-bts-lc15" "$configure_flags"</span><br><span> </span><br><span>diff --git a/contrib/jenkins_oct.sh b/contrib/jenkins_oct.sh</span><br><span>index efbd368..bd57dd1 100755</span><br><span>--- a/contrib/jenkins_oct.sh</span><br><span>+++ b/contrib/jenkins_oct.sh</span><br><span>@@ -14,7 +14,12 @@</span><br><span> cd "$deps"</span><br><span> osmo-layer1-headers.sh oct "$FIRMWARE_VERSION"</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-configure_flags="--enable-sanitize --with-octsdr-2g=$deps/layer1-headers/ --enable-octphy"</span><br><span style="color: hsl(120, 100%, 40%);">+configure_flags="\</span><br><span style="color: hsl(120, 100%, 40%);">+  --enable-sanitize \</span><br><span style="color: hsl(120, 100%, 40%);">+  --enable-werror \</span><br><span style="color: hsl(120, 100%, 40%);">+  --with-octsdr-2g=$deps/layer1-headers/ \</span><br><span style="color: hsl(120, 100%, 40%);">+  --enable-octphy \</span><br><span style="color: hsl(120, 100%, 40%);">+  "</span><br><span> </span><br><span> build_bts "osmo-bts-octphy" "$configure_flags"</span><br><span> </span><br><span>diff --git a/contrib/jenkins_oct_and_bts_trx.sh b/contrib/jenkins_oct_and_bts_trx.sh</span><br><span>index f68a9d3..049f5f7 100755</span><br><span>--- a/contrib/jenkins_oct_and_bts_trx.sh</span><br><span>+++ b/contrib/jenkins_oct_and_bts_trx.sh</span><br><span>@@ -16,6 +16,7 @@</span><br><span> osmo-layer1-headers.sh oct "$FIRMWARE_VERSION"</span><br><span> </span><br><span> configure_flags="\</span><br><span style="color: hsl(120, 100%, 40%);">+  --enable-werror \</span><br><span>   --with-osmo-pcu=$deps/osmo-pcu/include \</span><br><span>   --with-octsdr-2g=$deps/layer1-headers/ \</span><br><span>   --enable-octphy \</span><br><span>diff --git a/contrib/jenkins_sysmobts.sh b/contrib/jenkins_sysmobts.sh</span><br><span>index 7488419..d0d05ae 100755</span><br><span>--- a/contrib/jenkins_sysmobts.sh</span><br><span>+++ b/contrib/jenkins_sysmobts.sh</span><br><span>@@ -16,7 +16,12 @@</span><br><span> mkdir -p "$inst/include/sysmocom/femtobts"</span><br><span> ln -s $deps/layer1-headers/include/* "$inst/include/sysmocom/femtobts/"</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-configure_flags="--enable-sanitize --enable-sysmocom-bts --with-sysmobts=$inst/include/"</span><br><span style="color: hsl(120, 100%, 40%);">+configure_flags="\</span><br><span style="color: hsl(120, 100%, 40%);">+  --enable-sanitize \</span><br><span style="color: hsl(120, 100%, 40%);">+  --enable-werror \</span><br><span style="color: hsl(120, 100%, 40%);">+  --enable-sysmocom-bts \</span><br><span style="color: hsl(120, 100%, 40%);">+  --with-sysmobts=$inst/include/ \</span><br><span style="color: hsl(120, 100%, 40%);">+  "</span><br><span> </span><br><span> # This will not work for the femtobts</span><br><span> if [ $FIRMWARE_VERSION != "femtobts_v2.7" ]; then</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/9536">change 9536</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/9536"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bts </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: merged </div>
<div style="display:none"> Gerrit-Change-Id: Ib65056633697dafa63831fc2a480b798df550db6 </div>
<div style="display:none"> Gerrit-Change-Number: 9536 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: Vadim Yanitskiy <axilirator@gmail.com> </div>