<p>Neels Hofmeyr <strong>merged</strong> this change.</p><p><a href="https://gerrit.osmocom.org/11634">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  Neels Hofmeyr: Looks good to me, approved; Verified

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">deps: comment out osmo-trx<br><br>osmo-trx shouldn't run with address sanitizer. But omitting --enable-sanitize<br>from osmo-trx breaks 'make check', so then the whole top-level makefile will<br>end in error with sanitize.opts.<br><br>A user wanting to build osmo-trx can just uncomment that line.<br><br>Change-Id: I589f49652f8fa4e5becc7d7e63e6e4bc1a8b33bb<br>---<br>M 2G.deps<br>M 3G+2G.deps<br>2 files changed, 8 insertions(+), 2 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/2G.deps b/2G.deps</span><br><span>index cf2723a..c49589d 100644</span><br><span>--- a/2G.deps</span><br><span>+++ b/2G.deps</span><br><span>@@ -10,4 +10,7 @@</span><br><span> osmo-msc     libosmo-sccp osmo-mgw libsmpp34</span><br><span> osmo-bsc     libosmo-sccp osmo-mgw</span><br><span> osmo-sgsn      libosmo-sccp osmo-ggsn osmo-hlr</span><br><span style="color: hsl(0, 100%, 40%);">-osmo-trx libosmocore</span><br><span style="color: hsl(120, 100%, 40%);">+# osmo-trx can build with --enable-sanitize, but then won't work reliably.</span><br><span style="color: hsl(120, 100%, 40%);">+# When omitting --enable-sanitize from osmo-trx only, its 'make check' will fail.</span><br><span style="color: hsl(120, 100%, 40%);">+# So if you want osmo-trx, uncomment and use no sanitize.opts, or use LD_PRELOAD for osmo-trx binaries.</span><br><span style="color: hsl(120, 100%, 40%);">+#osmo-trx      libosmocore</span><br><span>diff --git a/3G+2G.deps b/3G+2G.deps</span><br><span>index b82e7c9..e5d53fa 100644</span><br><span>--- a/3G+2G.deps</span><br><span>+++ b/3G+2G.deps</span><br><span>@@ -13,4 +13,7 @@</span><br><span> osmo-bsc        libosmo-sccp osmo-mgw</span><br><span> osmo-sgsn      osmo-iuh osmo-ggsn osmo-hlr</span><br><span> osmo-sip-connector       libosmocore</span><br><span style="color: hsl(0, 100%, 40%);">-osmo-trx     libosmocore</span><br><span style="color: hsl(120, 100%, 40%);">+# osmo-trx can build with --enable-sanitize, but then won't work reliably.</span><br><span style="color: hsl(120, 100%, 40%);">+# When omitting --enable-sanitize from osmo-trx only, its 'make check' will fail.</span><br><span style="color: hsl(120, 100%, 40%);">+# So if you want osmo-trx, uncomment and use no sanitize.opts, or use LD_PRELOAD for osmo-trx binaries.</span><br><span style="color: hsl(120, 100%, 40%);">+#osmo-trx      libosmocore</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/11634">change 11634</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/11634"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-dev </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: merged </div>
<div style="display:none"> Gerrit-Change-Id: I589f49652f8fa4e5becc7d7e63e6e4bc1a8b33bb </div>
<div style="display:none"> Gerrit-Change-Number: 11634 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: osmith <osmith@sysmocom.de> </div>