* stuff is in the wrong directory, move to sbin * CFLAGS is abused for preprocessor, move to CPPFLAGS * add missing $(LIBGTPNL_CFLAGS) * ggsn includes libmnl.h but fails to PKG_CHECK for it; turns out it only uses the header and no defs, so kill that. * _DEPENDENCIES on files not generated by $this Makefile are pointless * must use .la files in _LDADD/_LIBADD whenever they exist * remove AC_PROG_CXX, the C++ is never used --- ggsn/Makefile.am | 15 +++++---------- gtp/Makefile.am | 4 +--- lib/Makefile.am | 12 +++++------- lib/gtp-kernel.c | 1 + sgsnemu/Makefile.am | 13 +++++-------- tests/gtp/Makefile.am | 2 +- tests/lib/Makefile.am | 4 ++-- 7 files changed, 20 insertions(+), 31 deletions(-) Index: osmo-ggsn-1.12.0/ggsn/Makefile.am =================================================================== --- osmo-ggsn-1.12.0.orig/ggsn/Makefile.am +++ osmo-ggsn-1.12.0/ggsn/Makefile.am @@ -1,24 +1,19 @@ -bin_PROGRAMS = osmo-ggsn +sbin_PROGRAMS = osmo-ggsn AM_LDFLAGS = @EXEC_LDFLAGS@ +AM_CPPFLAGS = -D_GNU_SOURCE -DSBINDIR='"$(sbindir)"' -I$(top_srcdir)/include $(LIBOSMOCORE_CFLAGS) $(LIBOSMOCTRL_CFLAGS) $(LIBOSMOVTY_CFLAGS) AM_CFLAGS = \ - -D_GNU_SOURCE \ -fno-builtin \ -Wall \ - -DSBINDIR='"$(sbindir)"' \ - -I$(top_srcdir)/include \ - $(LIBOSMOCORE_CFLAGS) \ - $(LIBOSMOCTRL_CFLAGS) \ - $(LIBOSMOVTY_CFLAGS) \ $(NULL) -osmo_ggsn_LDADD = @EXEC_LDADD@ -lgtp -L../gtp ../lib/libmisc.a $(LIBOSMOCORE_LIBS) $(LIBOSMOCTRL_LIBS) $(LIBOSMOVTY_LIBS) +osmo_ggsn_LDADD = @EXEC_LDADD@ ../gtp/libgtp.la ../lib/libmisc.la $(LIBOSMOCORE_LIBS) $(LIBOSMOCTRL_LIBS) $(LIBOSMOVTY_LIBS) if ENABLE_GTP_KERNEL -AM_CFLAGS += -DGTP_KERNEL $(LIBGTPNL_CFLAGS) +AM_CPPFLAGS += -DGTP_KERNEL $(LIBGTPNL_CFLAGS) osmo_ggsn_LDADD += $(LIBGTPNL_LIBS) endif -osmo_ggsn_DEPENDENCIES = ../gtp/libgtp.la ../lib/libmisc.a +osmo_ggsn_DEPENDENCIES = ../gtp/libgtp.la ../lib/libmisc.la osmo_ggsn_SOURCES = ggsn_main.c ggsn_vty.c ggsn.c ggsn.h sgsn.c sgsn.h pco.c pco.h Index: osmo-ggsn-1.12.0/gtp/Makefile.am =================================================================== --- osmo-ggsn-1.12.0.orig/gtp/Makefile.am +++ osmo-ggsn-1.12.0/gtp/Makefile.am @@ -6,12 +6,10 @@ LIBVERSION=10:0:0 lib_LTLIBRARIES = libgtp.la +AM_CPPFLAGS = -DSBINDIR='"$(sbindir)"' -I$(top_srcdir)/include $(LIBOSMOCORE_CFLAGS) AM_CFLAGS = \ -fno-builtin \ -Wall \ - -DSBINDIR='"$(sbindir)"' \ - -I$(top_srcdir)/include \ - $(LIBOSMOCORE_CFLAGS) \ $(NULL) libgtp_la_SOURCES = \ Index: osmo-ggsn-1.12.0/lib/Makefile.am =================================================================== --- osmo-ggsn-1.12.0.orig/lib/Makefile.am +++ osmo-ggsn-1.12.0/lib/Makefile.am @@ -1,4 +1,4 @@ -noinst_LIBRARIES = libmisc.a +noinst_LTLIBRARIES = libmisc.la noinst_HEADERS = \ checksum.h \ @@ -15,15 +15,13 @@ noinst_HEADERS = \ util.h \ $(NULL) +AM_CPPFLAGS = -DSBINDIR='"$(sbindir)"' -I$(top_srcdir)/include $(LIBOSMOCORE_CFLAGS) AM_CFLAGS = \ -fno-builtin \ -Wall \ - -DSBINDIR='"$(sbindir)"' \ - -I$(top_srcdir)/include \ - $(LIBOSMOCORE_CFLAGS) \ $(NULL) -libmisc_a_SOURCES = \ +libmisc_la_SOURCES = \ checksum.c \ debug.c \ getopt.c \ @@ -39,6 +37,6 @@ libmisc_a_SOURCES = \ $(NULL) if ENABLE_GTP_KERNEL -AM_CFLAGS += -DGTP_KERNEL $(LIBGTPNL_CFLAGS) -libmisc_a_SOURCES += gtp-kernel.c +AM_CPPFLAGS += -DGTP_KERNEL $(LIBGTPNL_CFLAGS) +libmisc_la_SOURCES += gtp-kernel.c endif Index: osmo-ggsn-1.12.0/lib/gtp-kernel.c =================================================================== --- osmo-ggsn-1.12.0.orig/lib/gtp-kernel.c +++ osmo-ggsn-1.12.0/lib/gtp-kernel.c @@ -54,6 +54,7 @@ static void pdp_debug(const char *prefix buf4, buf6, inet_ntoa(ia)); } +struct mnl_socket; static struct { int genl_id; struct mnl_socket *nl; Index: osmo-ggsn-1.12.0/sgsnemu/Makefile.am =================================================================== --- osmo-ggsn-1.12.0.orig/sgsnemu/Makefile.am +++ osmo-ggsn-1.12.0/sgsnemu/Makefile.am @@ -1,25 +1,22 @@ -bin_PROGRAMS = sgsnemu +sbin_PROGRAMS = sgsnemu AM_LDFLAGS = @EXEC_LDFLAGS@ +AM_CPPFLAGS = -D_GNU_SOURCE -DSBINDIR='"$(sbindir)"' -I$(top_srcdir)/include $(LIBOSMOCORE_CFLAGS) AM_CFLAGS = \ - -D_GNU_SOURCE \ -fno-builtin \ -Wall \ - -DSBINDIR='"$(sbindir)"' \ - -I$(top_srcdir)/include \ - $(LIBOSMOCORE_CFLAGS) \ $(NULL) -sgsnemu_LDADD = @EXEC_LDADD@ -lgtp -L../gtp ../lib/libmisc.a $(LIBOSMOCORE_LIBS) +sgsnemu_LDADD = @EXEC_LDADD@ ../gtp/libgtp.la ../lib/libmisc.la $(LIBOSMOCORE_LIBS) if ENABLE_GTP_KERNEL -AM_CFLAGS += \ +AM_CPPFLAGS += \ -DGTP_KERNEL \ $(LIBGTPNL_CFLAGS) \ $(NULL) sgsnemu_LDADD += $(LIBGTPNL_LIBS) endif -sgsnemu_DEPENDENCIES = ../gtp/libgtp.la ../lib/libmisc.a +sgsnemu_DEPENDENCIES = ../gtp/libgtp.la ../lib/libmisc.la sgsnemu_SOURCES = sgsnemu.c cmdline.c cmdline.h Index: osmo-ggsn-1.12.0/tests/gtp/Makefile.am =================================================================== --- osmo-ggsn-1.12.0.orig/tests/gtp/Makefile.am +++ osmo-ggsn-1.12.0/tests/gtp/Makefile.am @@ -24,7 +24,7 @@ queue_test_SOURCES = \ $(NULL) gtpie_test_LDADD = \ - $(top_builddir)/lib/debug.o \ + $(top_builddir)/lib/debug.lo \ $(top_builddir)/gtp/libgtp.la \ $(LIBOSMOCORE_LIBS) \ $(NULL) Index: osmo-ggsn-1.12.0/tests/lib/Makefile.am =================================================================== --- osmo-ggsn-1.12.0.orig/tests/lib/Makefile.am +++ osmo-ggsn-1.12.0/tests/lib/Makefile.am @@ -19,7 +19,7 @@ ippool_test_SOURCES = \ $(NULL) ippool_test_LDADD = \ - $(top_builddir)/lib/libmisc.a \ + $(top_builddir)/lib/libmisc.la \ $(LIBOSMOCORE_LIBS) \ $(NULL) @@ -28,6 +28,6 @@ in46a_test_SOURCES = \ $(NULL) in46a_test_LDADD = \ - $(top_builddir)/lib/libmisc.a \ + $(top_builddir)/lib/libmisc.la \ $(LIBOSMOCORE_LIBS) \ $(NULL)