Change in libosmo-netif[master]: Don't depend on libosmo-abis (by default)

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

laforge gerrit-no-reply at lists.osmocom.org
Sat Feb 13 08:12:38 UTC 2021


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

Change subject: Don't depend on libosmo-abis (by default)
......................................................................

Don't depend on libosmo-abis (by default)

The only reason we have a libosmo-netif -> libosmo-abis
dependency is the lapd examples whihc are built but not even installed.

Let's build those only if --enable-lapd-examples is specified at the
command line, and remove the dependency to libosmo-abis in all other
cases.

Change-Id: Ida8157cd9111b196e4bf08782c45d0e3d393f1c9
Closes: OS#4726
---
M configure.ac
M contrib/jenkins.sh
M contrib/libosmo-netif.spec.in
M debian/control
M examples/Makefile.am
5 files changed, 17 insertions(+), 7 deletions(-)

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



diff --git a/configure.ac b/configure.ac
index 36c2f49..1b2a459 100644
--- a/configure.ac
+++ b/configure.ac
@@ -91,8 +91,19 @@
 
 PKG_CHECK_MODULES(LIBOSMOCORE, libosmocore >= 1.3.0)
 PKG_CHECK_MODULES(LIBOSMOGSM, libosmogsm >= 1.3.0)
-dnl FIXME: We depend on libosmoabis by now until we can move LAPD code here
-PKG_CHECK_MODULES(LIBOSMOABIS, libosmoabis >= 0.6.0)
+
+AC_ARG_ENABLE([lapd_examples],
+	[AS_HELP_STRING(
+		[--enable-lapd-examples],
+		[Build some lapd examples]
+	)],
+	[lapd_examples=$enableval], [lapd_examples="no"])
+AS_IF([test "x$lapd_examples" = "xyes"], [
+	PKG_CHECK_MODULES(LIBOSMOABIS, libosmoabis >= 0.6.0)
+	AC_DEFINE([ENABLE_LAPD], [1], [Enable LAPD examples])
+])
+AM_CONDITIONAL(ENABLE_LAPD, test "x$lapd_examples" = "xyes")
+AC_SUBST(ENABLE_LAPD)
 
 AC_ARG_ENABLE([libsctp], [AS_HELP_STRING([--disable-libsctp], [Do not enable socket multiaddr APIs requiring libsctp])],
 	[ENABLE_LIBSCTP=$enableval], [ENABLE_LIBSCTP="yes"])
diff --git a/contrib/jenkins.sh b/contrib/jenkins.sh
index bf04643..543671c 100755
--- a/contrib/jenkins.sh
+++ b/contrib/jenkins.sh
@@ -35,7 +35,7 @@
 set -x
 
 autoreconf --install --force
-./configure --enable-sanitize --enable-werror
+./configure --enable-sanitize --enable-werror --enable-lapd-examples
 $MAKE $PARALLEL_MAKE
 $MAKE $PARALLEL_MAKE distcheck \
   || cat-testlogs.sh
diff --git a/contrib/libosmo-netif.spec.in b/contrib/libosmo-netif.spec.in
index 98d021b..95690a7 100644
--- a/contrib/libosmo-netif.spec.in
+++ b/contrib/libosmo-netif.spec.in
@@ -24,7 +24,6 @@
 BuildRequires:  libtool >= 2
 BuildRequires:  lksctp-tools-devel
 BuildRequires:  pkgconfig >= 0.20
-BuildRequires:  pkgconfig(libosmoabis) >= 0.6.0
 BuildRequires:  pkgconfig(libosmocore) >= 1.0.0
 BuildRequires:  pkgconfig(libosmogsm) >= 1.0.0
 
diff --git a/debian/control b/debian/control
index d242b48..8b76089 100644
--- a/debian/control
+++ b/debian/control
@@ -12,7 +12,6 @@
                git,
                doxygen,
                libosmocore-dev (>= 1.3.0),
-               libosmo-abis-dev (>= 0.6.0),
                pkg-config,
                libpcap0.8-dev,
                libsctp-dev
diff --git a/examples/Makefile.am b/examples/Makefile.am
index 4125243..6d74238 100644
--- a/examples/Makefile.am
+++ b/examples/Makefile.am
@@ -4,8 +4,6 @@
 
 noinst_PROGRAMS = ipa-stream-client		\
 		  ipa-stream-server		\
-		  lapd-over-datagram-user	\
-		  lapd-over-datagram-network	\
 		  stream-client			\
 		  stream-server			\
 		  rs232-read			\
@@ -27,6 +25,8 @@
 ipa_stream_server_LDADD = $(top_builddir)/src/libosmonetif.la \
 			  $(LIBOSMOCORE_LIBS) $(LIBOSMOGSM_LIBS)
 
+if ENABLE_LAPD
+noinst_PROGRAMS += lapd-over-datagram-user lapd-over-datagram-network
 lapd_over_datagram_user_SOURCES = lapd-over-datagram-user.c
 lapd_over_datagram_user_LDADD = $(top_builddir)/src/libosmonetif.la \
 				  $(LIBOSMOCORE_LIBS) $(LIBOSMOGSM_LIBS) \
@@ -36,6 +36,7 @@
 lapd_over_datagram_network_LDADD = $(top_builddir)/src/libosmonetif.la \
 				  $(LIBOSMOCORE_LIBS) $(LIBOSMOGSM_LIBS) \
 				  $(LIBOSMOABIS_LIBS)
+endif
 
 stream_client_SOURCES = stream-client.c
 stream_client_LDADD = $(top_builddir)/src/libosmonetif.la \

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

Gerrit-Project: libosmo-netif
Gerrit-Branch: master
Gerrit-Change-Id: Ida8157cd9111b196e4bf08782c45d0e3d393f1c9
Gerrit-Change-Number: 22848
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <laforge at osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge at osmocom.org>
Gerrit-Reviewer: pespin <pespin at sysmocom.de>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20210213/c2b1c6dc/attachment.htm>


More information about the gerrit-log mailing list