laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/libosmo-pfcp/+/30256 )

Change subject: pfcp_test: fix order of dependencies
......................................................................

pfcp_test: fix order of dependencies

When LIBOSMOCORE_LIBS comes before locally built libs, for some reason
libtool puts system lib/ first in LD_LIBRARY_PATH, leading to ABI
breakage in pfcp_test when an older libosmo-pfcp is still installed.

Change-Id: I63880c1b4f326687ce9a35584a55742c1016d72b
---
M tests/libosmo-pfcp/Makefile.am
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  laforge: Looks good to me, approved
  pespin: Looks good to me, but someone else must approve
  osmith: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/tests/libosmo-pfcp/Makefile.am b/tests/libosmo-pfcp/Makefile.am
index b393d52..e9ff6d3 100644
--- a/tests/libosmo-pfcp/Makefile.am
+++ b/tests/libosmo-pfcp/Makefile.am
@@ -22,9 +22,9 @@
        $(NULL)

 pfcp_test_LDADD = \
-       $(LIBOSMOCORE_LIBS) \
        $(top_builddir)/src/libosmo-pfcp/libosmo-pfcp.la \
        $(top_builddir)/src/libosmo-gtlv/libosmo-gtlv.la \
+       $(LIBOSMOCORE_LIBS) \
        $(NULL)

 .PHONY: update_exp

--
To view, visit https://gerrit.osmocom.org/c/libosmo-pfcp/+/30256
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmo-pfcp
Gerrit-Branch: master
Gerrit-Change-Id: I63880c1b4f326687ce9a35584a55742c1016d72b
Gerrit-Change-Number: 30256
Gerrit-PatchSet: 2
Gerrit-Owner: neels <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: osmith <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-MessageType: merged

Reply via email to