<p>laforge <strong>submitted</strong> this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/20448">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  Jenkins Builder: Verified
  pespin: Looks good to me, but someone else must approve
  fixeria: Looks good to me, but someone else must approve
  laforge: Looks good to me, approved

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">regen-makefile.sh: Put libraries in LINUX_LIBS, not LDFLAGS<br><br>Additional libraries to be linked should be in LINUX_LIBS (appended at<br>the end of the linker command), not part of LDFLAGS (prepended to<br>the beginning of the linker command).<br><br>On binutils 2.35.1 / Debian unstable, without this patch, I get<br><br>/usr/bin/ld: IPL4asp_PT.so: undefined reference to `sctp_bindx'<br>/usr/bin/ld: IPL4asp_PT.so: undefined reference to `sctp_connectx'<br><br>which is resolved by this patch<br><br>Change-Id: I8a339076f445e3c650e407ae982c7c2dc4a760b2<br>---<br>M mme/regen_makefile.sh<br>M msc/regen_makefile.sh<br>M regen-makefile.sh<br>M sgsn/regen_makefile.sh<br>4 files changed, 5 insertions(+), 4 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/mme/regen_makefile.sh b/mme/regen_makefile.sh</span><br><span>index 2d3bf59..cb04777 100755</span><br><span>--- a/mme/regen_makefile.sh</span><br><span>+++ b/mme/regen_makefile.sh</span><br><span>@@ -6,4 +6,4 @@</span><br><span> </span><br><span> ../regen-makefile.sh MME_Tests.ttcn $FILES</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-sed -i -e 's/^LINUX_LIBS = -lxml2/LINUX_LIBS = -lxml2 -lfftranscode -lgnutls/' Makefile</span><br><span style="color: hsl(120, 100%, 40%);">+sed -i -e 's/^LINUX_LIBS = -lxml2 -lsctp/LINUX_LIBS = -lxml2 -lsctp -lfftranscode/' Makefile</span><br><span>diff --git a/msc/regen_makefile.sh b/msc/regen_makefile.sh</span><br><span>index 9278530..6ee3ef5 100755</span><br><span>--- a/msc/regen_makefile.sh</span><br><span>+++ b/msc/regen_makefile.sh</span><br><span>@@ -6,4 +6,4 @@</span><br><span> </span><br><span> ../regen-makefile.sh MSC_Tests.ttcn $FILES</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-sed -i -e 's/^LINUX_LIBS = -lxml2/LINUX_LIBS = -lxml2 -lfftranscode/' Makefile</span><br><span style="color: hsl(120, 100%, 40%);">+sed -i -e 's/^LINUX_LIBS = -lxml2 -lsctp/LINUX_LIBS = -lxml2 -lsctp -lfftranscode/' Makefile</span><br><span>diff --git a/regen-makefile.sh b/regen-makefile.sh</span><br><span>index f6e9538..086c124 100755</span><br><span>--- a/regen-makefile.sh</span><br><span>+++ b/regen-makefile.sh</span><br><span>@@ -45,7 +45,8 @@</span><br><span> TITAN_VERSION=$(ttcn3_makefilegen -v 2>&1 |grep "Product number" |cut --delimiter="/" -f 2-| sed -e "s/[A-Z ]//g")</span><br><span> </span><br><span> sed -i -e 's/# TTCN3_DIR = /TTCN3_DIR = \/usr/' Makefile</span><br><span style="color: hsl(0, 100%, 40%);">-sed -i -e 's/LDFLAGS = /LDFLAGS = -L \/usr\/lib\/titan -lsctp /' Makefile</span><br><span style="color: hsl(120, 100%, 40%);">+sed -i -e 's/LDFLAGS = /LDFLAGS = -L \/usr\/lib\/titan/' Makefile</span><br><span style="color: hsl(120, 100%, 40%);">+sed -i -e 's/LINUX_LIBS = -lxml2/LINUX_LIBS = -lxml2 -lsctp/' Makefile</span><br><span> #sed -i -e 's/TTCN3_LIB = ttcn3-parallel/TTCN3_LIB = ttcn3/' Makefile</span><br><span> </span><br><span> # The -DMAKEDEPEND_RUN is a workaround for Debian packaging issue,</span><br><span>diff --git a/sgsn/regen_makefile.sh b/sgsn/regen_makefile.sh</span><br><span>index 1d1411e..55ff26f 100755</span><br><span>--- a/sgsn/regen_makefile.sh</span><br><span>+++ b/sgsn/regen_makefile.sh</span><br><span>@@ -6,4 +6,4 @@</span><br><span> </span><br><span> ../regen-makefile.sh SGSN_Tests.ttcn $FILES</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-sed -i -e 's/^LINUX_LIBS = -lxml2/LINUX_LIBS = -lxml2 -lfftranscode/' Makefile</span><br><span style="color: hsl(120, 100%, 40%);">+sed -i -e 's/^LINUX_LIBS = -lxml2 -lsctp/LINUX_LIBS = -lxml2 -lsctp -lfftranscode/' Makefile</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/20448">change 20448</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/c/osmo-ttcn3-hacks/+/20448"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-ttcn3-hacks </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I8a339076f445e3c650e407ae982c7c2dc4a760b2 </div>
<div style="display:none"> Gerrit-Change-Number: 20448 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: merged </div>