netfilter-devel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* conntrack-tools: build system cleanups
@ 2010-10-30 22:49 Jan Engelhardt
  2010-10-30 22:49 ` [PATCH 1/7] build: use autoconf-suggested naming of files Jan Engelhardt
                   ` (7 more replies)
  0 siblings, 8 replies; 12+ messages in thread
From: Jan Engelhardt @ 2010-10-30 22:49 UTC (permalink / raw)
  To: pablo; +Cc: netfilter-devel


The following changes since commit 31ae56a2af1381459e4ec38c31ab810ffa410cf9:

  conntrack: allow to listen to all kind of expectation events (2010-10-12 12:56:01 +0200)

are available in the git repository at:
  git://dev.medozas.de/conntrack-tools master

Jan Engelhardt (7):
      build: use autoconf-suggested naming of files
      build: use modern call syntax for AM_INIT_AUTOMAKE
      build: drop unused $(all_includes)
      build: remove statements without effect
      build: remove unused $(all_libraries)
      build: no need for error message in PKG_CHECK_MODULES
      Add .gitignore files

 .gitignore                   |   18 ++++++++++++++++++
 Make_global.am               |    5 +++--
 Makefile.am                  |    7 -------
 configure.in => configure.ac |   31 +++++--------------------------
 src/.gitignore               |    6 ++++++
 src/Makefile.am              |    5 ++---
 6 files changed, 34 insertions(+), 38 deletions(-)
 create mode 100644 .gitignore
 rename configure.in => configure.ac (75%)
 create mode 100644 src/.gitignore

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [PATCH 1/7] build: use autoconf-suggested naming of files
  2010-10-30 22:49 conntrack-tools: build system cleanups Jan Engelhardt
@ 2010-10-30 22:49 ` Jan Engelhardt
  2010-10-30 22:49 ` [PATCH 2/7] build: use modern call syntax for AM_INIT_AUTOMAKE Jan Engelhardt
                   ` (6 subsequent siblings)
  7 siblings, 0 replies; 12+ messages in thread
From: Jan Engelhardt @ 2010-10-30 22:49 UTC (permalink / raw)
  To: pablo; +Cc: netfilter-devel

Signed-off-by: Jan Engelhardt <jengelh@medozas.de>
---
 configure.ac |  135 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 configure.in |  135 ----------------------------------------------------------
 2 files changed, 135 insertions(+), 135 deletions(-)
 create mode 100644 configure.ac
 delete mode 100644 configure.in

diff --git a/configure.ac b/configure.ac
new file mode 100644
index 0000000..88faede
--- /dev/null
+++ b/configure.ac
@@ -0,0 +1,135 @@
+AC_INIT(conntrack-tools, 0.9.15, pablo@netfilter.org)
+
+AC_CANONICAL_SYSTEM
+
+AC_CONFIG_MACRO_DIR([m4])
+AM_INIT_AUTOMAKE([-Wall])
+
+AC_PROG_CC
+AM_PROG_LIBTOOL
+AC_PROG_INSTALL
+AC_PROG_LN_S
+AM_PROG_LEX
+AC_PROG_YACC
+
+case $target in
+*-*-linux*) ;;
+*) AC_MSG_ERROR([Linux only, dude!]);;
+esac
+
+dnl Dependencies
+LIBNFNETLINK_REQUIRED=1.0.0
+LIBNETFILTER_CONNTRACK_REQUIRED=0.0.102
+
+AC_CHECK_PROG(HAVE_PKG_CONFIG, pkg-config, yes)
+if test "x$HAVE_PKG_CONFIG" = "x"
+then
+	echo "*** Error: No suitable pkg-config found. ***"
+	echo "    Please install the 'pkg-config' package."
+	exit 1
+fi
+
+PKG_CHECK_MODULES(LIBNFNETLINK, libnfnetlink >= $LIBNFNETLINK_REQUIRED,,
+        AC_MSG_ERROR(Cannot find libnfnetlink >= $LIBNFNETLINK_REQUIRED))
+
+PKG_CHECK_MODULES(LIBNETFILTER_CONNTRACK, libnetfilter_conntrack >= $LIBNETFILTER_CONNTRACK_REQUIRED,,
+        AC_MSG_ERROR(Cannot find libnetfilter_conntrack >= $LIBNETFILTER_CONNTRACK_REQUIRED))
+
+AC_CHECK_PROGS(XYACC,$YACC bison yacc,none)
+if test "$XYACC" = "none"
+then
+	echo "*** Error: No suitable bison/yacc found. ***"
+	echo "    Please install the 'bison' package."
+	exit 1
+fi
+AC_CHECK_PROGS(XLEX,$LEX flex lex,none)
+if test "$XLEX" = "none"
+then
+	echo "*** Error: No suitable flex/lex found. ***"
+	echo "    Please install the 'flex' package."
+	exit 1
+fi
+
+AC_MSG_CHECKING(flex version)
+flex_version=`$LEX --version | sed 's/version//g' | awk '/flex/ {print $2}'`
+flex_major=`echo $flex_version| cut -d . -f 1`
+flex_minor=`echo $flex_version| cut -d . -f 2`
+flex_rev=`echo $flex_version| cut -d . -f 3`
+ 
+if test "$flex_major" -eq "2" && test "$flex_minor" -eq "5" && test "$flex_rev" -ge "33"; then
+	AC_MSG_RESULT([$flex_version. OK])
+else
+	AC_MSG_WARN([flex version $flex_version found.
+	Version 2.5.33 or greater is required. You may experience problems
+	while compilating the conntrack-tools. Please, consider to upgrade 
+	flex.])
+fi
+
+AC_CHECK_HEADERS([linux/capability.h],, [AC_MSG_ERROR([Cannot find linux/capabibility.h])])
+
+# Checks for libraries.
+# FIXME: Replace `main' with a function in `-lc':
+dnl AC_CHECK_LIB([c], [main])
+# FIXME: Replace `main' with a function in `-ldl':
+
+AC_CHECK_LIB([nfnetlink], [nfnl_query] ,,,[-lnfnetlink])
+AC_CHECK_LIB([netfilter_conntrack], [nfct_query] ,,,[-lnetfilter_conntrack])
+
+AC_CHECK_HEADERS(arpa/inet.h)
+dnl check for inet_pton
+AC_CHECK_FUNCS(inet_pton)
+dnl Some systems have it, but not IPv6
+if test "$ac_cv_func_inet_pton" = "yes" ; then
+AC_MSG_CHECKING(if inet_pton supports IPv6)
+AC_TRY_RUN(
+   [
+#ifdef HAVE_SYS_TYPES_H
+#include <sys/types.h>
+#endif
+#ifdef HAVE_SYS_SOCKET_H
+#include <sys/socket.h>
+#endif
+#ifdef HAVE_NETINET_IN_H
+#include <netinet/in.h>
+#endif
+#ifdef HAVE_ARPA_INET_H
+#include <arpa/inet.h>
+#endif
+int main()
+  {
+     struct in6_addr addr6;
+     if (inet_pton(AF_INET6, "::1", &addr6) < 1)
+        exit(1);
+     else
+        exit(0);
+  }
+  ], [ AC_MSG_RESULT(yes)
+       AC_DEFINE_UNQUOTED(HAVE_INET_PTON_IPV6, 1, [Define to 1 if inet_pton supports IPv6.])
+  ], AC_MSG_RESULT(no), AC_MSG_RESULT(no))
+fi
+
+# Checks for header files.
+dnl AC_HEADER_STDC
+dnl AC_CHECK_HEADERS([netinet/in.h stdlib.h])
+
+# Checks for typedefs, structures, and compiler characteristics.
+dnl AC_C_CONST
+dnl AC_C_INLINE
+
+# Checks for library functions.
+dnl AC_FUNC_MALLOC
+dnl AC_FUNC_VPRINTF
+dnl AC_CHECK_FUNCS([memset])
+
+dnl AC_CONFIG_FILES([Makefile
+dnl                  debug/Makefile
+dnl                  debug/src/Makefile
+dnl                  extensions/Makefile
+dnl                  src/Makefile])
+
+CFLAGS="$CFLAGS $LIBNETFILTER_CONNTRACK_CFLAGS"
+
+AC_SUBST(LIBNETFILTER_CONNTRACK_LIBS)
+
+AC_CONFIG_FILES([Makefile src/Makefile include/Makefile extensions/Makefile])
+AC_OUTPUT
diff --git a/configure.in b/configure.in
deleted file mode 100644
index 88faede..0000000
--- a/configure.in
+++ /dev/null
@@ -1,135 +0,0 @@
-AC_INIT(conntrack-tools, 0.9.15, pablo@netfilter.org)
-
-AC_CANONICAL_SYSTEM
-
-AC_CONFIG_MACRO_DIR([m4])
-AM_INIT_AUTOMAKE([-Wall])
-
-AC_PROG_CC
-AM_PROG_LIBTOOL
-AC_PROG_INSTALL
-AC_PROG_LN_S
-AM_PROG_LEX
-AC_PROG_YACC
-
-case $target in
-*-*-linux*) ;;
-*) AC_MSG_ERROR([Linux only, dude!]);;
-esac
-
-dnl Dependencies
-LIBNFNETLINK_REQUIRED=1.0.0
-LIBNETFILTER_CONNTRACK_REQUIRED=0.0.102
-
-AC_CHECK_PROG(HAVE_PKG_CONFIG, pkg-config, yes)
-if test "x$HAVE_PKG_CONFIG" = "x"
-then
-	echo "*** Error: No suitable pkg-config found. ***"
-	echo "    Please install the 'pkg-config' package."
-	exit 1
-fi
-
-PKG_CHECK_MODULES(LIBNFNETLINK, libnfnetlink >= $LIBNFNETLINK_REQUIRED,,
-        AC_MSG_ERROR(Cannot find libnfnetlink >= $LIBNFNETLINK_REQUIRED))
-
-PKG_CHECK_MODULES(LIBNETFILTER_CONNTRACK, libnetfilter_conntrack >= $LIBNETFILTER_CONNTRACK_REQUIRED,,
-        AC_MSG_ERROR(Cannot find libnetfilter_conntrack >= $LIBNETFILTER_CONNTRACK_REQUIRED))
-
-AC_CHECK_PROGS(XYACC,$YACC bison yacc,none)
-if test "$XYACC" = "none"
-then
-	echo "*** Error: No suitable bison/yacc found. ***"
-	echo "    Please install the 'bison' package."
-	exit 1
-fi
-AC_CHECK_PROGS(XLEX,$LEX flex lex,none)
-if test "$XLEX" = "none"
-then
-	echo "*** Error: No suitable flex/lex found. ***"
-	echo "    Please install the 'flex' package."
-	exit 1
-fi
-
-AC_MSG_CHECKING(flex version)
-flex_version=`$LEX --version | sed 's/version//g' | awk '/flex/ {print $2}'`
-flex_major=`echo $flex_version| cut -d . -f 1`
-flex_minor=`echo $flex_version| cut -d . -f 2`
-flex_rev=`echo $flex_version| cut -d . -f 3`
- 
-if test "$flex_major" -eq "2" && test "$flex_minor" -eq "5" && test "$flex_rev" -ge "33"; then
-	AC_MSG_RESULT([$flex_version. OK])
-else
-	AC_MSG_WARN([flex version $flex_version found.
-	Version 2.5.33 or greater is required. You may experience problems
-	while compilating the conntrack-tools. Please, consider to upgrade 
-	flex.])
-fi
-
-AC_CHECK_HEADERS([linux/capability.h],, [AC_MSG_ERROR([Cannot find linux/capabibility.h])])
-
-# Checks for libraries.
-# FIXME: Replace `main' with a function in `-lc':
-dnl AC_CHECK_LIB([c], [main])
-# FIXME: Replace `main' with a function in `-ldl':
-
-AC_CHECK_LIB([nfnetlink], [nfnl_query] ,,,[-lnfnetlink])
-AC_CHECK_LIB([netfilter_conntrack], [nfct_query] ,,,[-lnetfilter_conntrack])
-
-AC_CHECK_HEADERS(arpa/inet.h)
-dnl check for inet_pton
-AC_CHECK_FUNCS(inet_pton)
-dnl Some systems have it, but not IPv6
-if test "$ac_cv_func_inet_pton" = "yes" ; then
-AC_MSG_CHECKING(if inet_pton supports IPv6)
-AC_TRY_RUN(
-   [
-#ifdef HAVE_SYS_TYPES_H
-#include <sys/types.h>
-#endif
-#ifdef HAVE_SYS_SOCKET_H
-#include <sys/socket.h>
-#endif
-#ifdef HAVE_NETINET_IN_H
-#include <netinet/in.h>
-#endif
-#ifdef HAVE_ARPA_INET_H
-#include <arpa/inet.h>
-#endif
-int main()
-  {
-     struct in6_addr addr6;
-     if (inet_pton(AF_INET6, "::1", &addr6) < 1)
-        exit(1);
-     else
-        exit(0);
-  }
-  ], [ AC_MSG_RESULT(yes)
-       AC_DEFINE_UNQUOTED(HAVE_INET_PTON_IPV6, 1, [Define to 1 if inet_pton supports IPv6.])
-  ], AC_MSG_RESULT(no), AC_MSG_RESULT(no))
-fi
-
-# Checks for header files.
-dnl AC_HEADER_STDC
-dnl AC_CHECK_HEADERS([netinet/in.h stdlib.h])
-
-# Checks for typedefs, structures, and compiler characteristics.
-dnl AC_C_CONST
-dnl AC_C_INLINE
-
-# Checks for library functions.
-dnl AC_FUNC_MALLOC
-dnl AC_FUNC_VPRINTF
-dnl AC_CHECK_FUNCS([memset])
-
-dnl AC_CONFIG_FILES([Makefile
-dnl                  debug/Makefile
-dnl                  debug/src/Makefile
-dnl                  extensions/Makefile
-dnl                  src/Makefile])
-
-CFLAGS="$CFLAGS $LIBNETFILTER_CONNTRACK_CFLAGS"
-
-AC_SUBST(LIBNETFILTER_CONNTRACK_LIBS)
-
-AC_CONFIG_FILES([Makefile src/Makefile include/Makefile extensions/Makefile])
-AC_OUTPUT
-- 
1.7.1


^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 2/7] build: use modern call syntax for AM_INIT_AUTOMAKE
  2010-10-30 22:49 conntrack-tools: build system cleanups Jan Engelhardt
  2010-10-30 22:49 ` [PATCH 1/7] build: use autoconf-suggested naming of files Jan Engelhardt
@ 2010-10-30 22:49 ` Jan Engelhardt
  2010-10-30 22:49 ` [PATCH 3/7] build: drop unused $(all_includes) Jan Engelhardt
                   ` (5 subsequent siblings)
  7 siblings, 0 replies; 12+ messages in thread
From: Jan Engelhardt @ 2010-10-30 22:49 UTC (permalink / raw)
  To: pablo; +Cc: netfilter-devel

automake options also need to definitely go into configure.ac,
otherwise they only apply to a single directory.

Signed-off-by: Jan Engelhardt <jengelh@medozas.de>
---
 Makefile.am  |    3 ---
 configure.ac |    2 +-
 2 files changed, 1 insertions(+), 4 deletions(-)

diff --git a/Makefile.am b/Makefile.am
index 70c1bd0..23f646c 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -1,9 +1,6 @@
 include Make_global.am
 
-# not a GNU package. You can remove this line, if
-# have all needed files, that a GNU package needs
 ACLOCAL_AMFLAGS  = -I m4
-AUTOMAKE_OPTIONS = foreign dist-bzip2 1.6
 
 man_MANS = conntrack.8 conntrackd.8
 EXTRA_DIST = $(man_MANS) Make_global.am doc m4
diff --git a/configure.ac b/configure.ac
index 88faede..f840a00 100644
--- a/configure.ac
+++ b/configure.ac
@@ -3,7 +3,7 @@ AC_INIT(conntrack-tools, 0.9.15, pablo@netfilter.org)
 AC_CANONICAL_SYSTEM
 
 AC_CONFIG_MACRO_DIR([m4])
-AM_INIT_AUTOMAKE([-Wall])
+AM_INIT_AUTOMAKE([-Wall foreign subdir-objects dist-bzip2 1.6])
 
 AC_PROG_CC
 AM_PROG_LIBTOOL
-- 
1.7.1


^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 3/7] build: drop unused $(all_includes)
  2010-10-30 22:49 conntrack-tools: build system cleanups Jan Engelhardt
  2010-10-30 22:49 ` [PATCH 1/7] build: use autoconf-suggested naming of files Jan Engelhardt
  2010-10-30 22:49 ` [PATCH 2/7] build: use modern call syntax for AM_INIT_AUTOMAKE Jan Engelhardt
@ 2010-10-30 22:49 ` Jan Engelhardt
  2010-10-30 22:49 ` [PATCH 4/7] build: remove statements without effect Jan Engelhardt
                   ` (4 subsequent siblings)
  7 siblings, 0 replies; 12+ messages in thread
From: Jan Engelhardt @ 2010-10-30 22:49 UTC (permalink / raw)
  To: pablo; +Cc: netfilter-devel

Signed-off-by: Jan Engelhardt <jengelh@medozas.de>
---
 Make_global.am |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/Make_global.am b/Make_global.am
index 810253a..4e9794e 100644
--- a/Make_global.am
+++ b/Make_global.am
@@ -1,4 +1,4 @@
-AM_CPPFLAGS=$(all_includes) -I$(top_srcdir)/include
+AM_CPPFLAGS = -I$(top_srcdir)/include
 
 AM_CFLAGS = -std=gnu99 -W -Wall \
 	-Wmissing-prototypes -Wwrite-strings -Wcast-qual -Wfloat-equal -Wshadow -Wpointer-arith -Wbad-function-cast -Wsign-compare -Waggregate-return -Wmissing-declarations -Wredundant-decls -Wnested-externs -Winline -Wstrict-prototypes -Wundef \
-- 
1.7.1


^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 4/7] build: remove statements without effect
  2010-10-30 22:49 conntrack-tools: build system cleanups Jan Engelhardt
                   ` (2 preceding siblings ...)
  2010-10-30 22:49 ` [PATCH 3/7] build: drop unused $(all_includes) Jan Engelhardt
@ 2010-10-30 22:49 ` Jan Engelhardt
  2010-10-30 22:49 ` [PATCH 5/7] build: remove unused $(all_libraries) Jan Engelhardt
                   ` (3 subsequent siblings)
  7 siblings, 0 replies; 12+ messages in thread
From: Jan Engelhardt @ 2010-10-30 22:49 UTC (permalink / raw)
  To: pablo; +Cc: netfilter-devel

Signed-off-by: Jan Engelhardt <jengelh@medozas.de>
---
 Makefile.am  |    4 ----
 configure.ac |    4 ++--
 2 files changed, 2 insertions(+), 6 deletions(-)

diff --git a/Makefile.am b/Makefile.am
index 23f646c..afb4595 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -9,9 +9,5 @@ SUBDIRS   = extensions src
 DIST_SUBDIRS = include src extensions
 LIBS = @LIBNETFILTER_CONNTRACK_LIBS@
 
-$(OBJECTS): libtool
-libtool: $(LIBTOOL_DEPS)
-	$(SHELL) ./config.status --recheck
-
 dist-hook:
 	rm -rf `find $(distdir)/doc -name .svn`
diff --git a/configure.ac b/configure.ac
index f840a00..e068798 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1,6 +1,6 @@
 AC_INIT(conntrack-tools, 0.9.15, pablo@netfilter.org)
 
-AC_CANONICAL_SYSTEM
+AC_CANONICAL_HOST
 
 AC_CONFIG_MACRO_DIR([m4])
 AM_INIT_AUTOMAKE([-Wall foreign subdir-objects dist-bzip2 1.6])
@@ -12,7 +12,7 @@ AC_PROG_LN_S
 AM_PROG_LEX
 AC_PROG_YACC
 
-case $target in
+case "$host" in
 *-*-linux*) ;;
 *) AC_MSG_ERROR([Linux only, dude!]);;
 esac
-- 
1.7.1


^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 5/7] build: remove unused $(all_libraries)
  2010-10-30 22:49 conntrack-tools: build system cleanups Jan Engelhardt
                   ` (3 preceding siblings ...)
  2010-10-30 22:49 ` [PATCH 4/7] build: remove statements without effect Jan Engelhardt
@ 2010-10-30 22:49 ` Jan Engelhardt
  2010-10-30 22:49 ` [PATCH 6/7] build: no need for error message in PKG_CHECK_MODULES Jan Engelhardt
                   ` (2 subsequent siblings)
  7 siblings, 0 replies; 12+ messages in thread
From: Jan Engelhardt @ 2010-10-30 22:49 UTC (permalink / raw)
  To: pablo; +Cc: netfilter-devel

Signed-off-by: Jan Engelhardt <jengelh@medozas.de>
---
 src/Makefile.am |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/Makefile.am b/src/Makefile.am
index 76f0e73..0e9af2e 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -8,7 +8,7 @@ sbin_PROGRAMS = conntrack conntrackd
 
 conntrack_SOURCES = conntrack.c
 conntrack_LDADD = ../extensions/libct_proto_tcp.la ../extensions/libct_proto_udp.la ../extensions/libct_proto_udplite.la ../extensions/libct_proto_icmp.la ../extensions/libct_proto_icmpv6.la ../extensions/libct_proto_sctp.la ../extensions/libct_proto_dccp.la ../extensions/libct_proto_gre.la ../extensions/libct_proto_unknown.la
-conntrack_LDFLAGS = $(all_libraries) @LIBNETFILTER_CONNTRACK_LIBS@
+conntrack_LDFLAGS = @LIBNETFILTER_CONNTRACK_LIBS@
 
 conntrackd_SOURCES = alarm.c main.c run.c hash.c queue.c rbtree.c \
 		    local.c log.c mcast.c udp.c netlink.c vector.c \
@@ -28,6 +28,6 @@ conntrackd_SOURCES = alarm.c main.c run.c hash.c queue.c rbtree.c \
 # yacc and lex generate dirty code
 read_config_yy.o read_config_lex.o: AM_CFLAGS += -Wno-missing-prototypes -Wno-missing-declarations -Wno-implicit-function-declaration -Wno-nested-externs -Wno-undef -Wno-redundant-decls
 
-conntrackd_LDFLAGS = $(all_libraries) @LIBNETFILTER_CONNTRACK_LIBS@
+conntrackd_LDFLAGS = @LIBNETFILTER_CONNTRACK_LIBS@
 
 EXTRA_DIST = read_config_yy.h
-- 
1.7.1


^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 6/7] build: no need for error message in PKG_CHECK_MODULES
  2010-10-30 22:49 conntrack-tools: build system cleanups Jan Engelhardt
                   ` (4 preceding siblings ...)
  2010-10-30 22:49 ` [PATCH 5/7] build: remove unused $(all_libraries) Jan Engelhardt
@ 2010-10-30 22:49 ` Jan Engelhardt
  2010-10-30 22:49 ` [PATCH 7/7] Add .gitignore files Jan Engelhardt
  2010-11-01 21:37 ` conntrack-tools: build system cleanups Pablo Neira Ayuso
  7 siblings, 0 replies; 12+ messages in thread
From: Jan Engelhardt @ 2010-10-30 22:49 UTC (permalink / raw)
  To: pablo; +Cc: netfilter-devel

PKG_CHECK_MODULES already produces its own (and more verbose)
messsage when a module cannot be found.

Mucking around with CFLAGS and LIBS is also not needed since
pkgconfig takes care of providing variables, so let's use them in
Makefile.am.

Signed-off-by: Jan Engelhardt <jengelh@medozas.de>
---
 Make_global.am  |    3 ++-
 configure.ac    |   25 ++-----------------------
 src/Makefile.am |    5 ++---
 3 files changed, 6 insertions(+), 27 deletions(-)

diff --git a/Make_global.am b/Make_global.am
index 4e9794e..e8f603a 100644
--- a/Make_global.am
+++ b/Make_global.am
@@ -2,4 +2,5 @@ AM_CPPFLAGS = -I$(top_srcdir)/include
 
 AM_CFLAGS = -std=gnu99 -W -Wall \
 	-Wmissing-prototypes -Wwrite-strings -Wcast-qual -Wfloat-equal -Wshadow -Wpointer-arith -Wbad-function-cast -Wsign-compare -Waggregate-return -Wmissing-declarations -Wredundant-decls -Wnested-externs -Winline -Wstrict-prototypes -Wundef \
-	-Wno-unused-parameter
+	-Wno-unused-parameter ${LIBNFNETLINK_CFLAGS} \
+	${LIBNETFILTER_CONNTRACK_CFLAGS}
diff --git a/configure.ac b/configure.ac
index e068798..37fbeee 100644
--- a/configure.ac
+++ b/configure.ac
@@ -18,22 +18,8 @@ case "$host" in
 esac
 
 dnl Dependencies
-LIBNFNETLINK_REQUIRED=1.0.0
-LIBNETFILTER_CONNTRACK_REQUIRED=0.0.102
-
-AC_CHECK_PROG(HAVE_PKG_CONFIG, pkg-config, yes)
-if test "x$HAVE_PKG_CONFIG" = "x"
-then
-	echo "*** Error: No suitable pkg-config found. ***"
-	echo "    Please install the 'pkg-config' package."
-	exit 1
-fi
-
-PKG_CHECK_MODULES(LIBNFNETLINK, libnfnetlink >= $LIBNFNETLINK_REQUIRED,,
-        AC_MSG_ERROR(Cannot find libnfnetlink >= $LIBNFNETLINK_REQUIRED))
-
-PKG_CHECK_MODULES(LIBNETFILTER_CONNTRACK, libnetfilter_conntrack >= $LIBNETFILTER_CONNTRACK_REQUIRED,,
-        AC_MSG_ERROR(Cannot find libnetfilter_conntrack >= $LIBNETFILTER_CONNTRACK_REQUIRED))
+PKG_CHECK_MODULES([LIBNFNETLINK], [libnfnetlink >= 1.0.0])
+PKG_CHECK_MODULES([LIBNETFILTER_CONNTRACK], [libnetfilter_conntrack >= 0.0.102])
 
 AC_CHECK_PROGS(XYACC,$YACC bison yacc,none)
 if test "$XYACC" = "none"
@@ -72,9 +58,6 @@ AC_CHECK_HEADERS([linux/capability.h],, [AC_MSG_ERROR([Cannot find linux/capabib
 dnl AC_CHECK_LIB([c], [main])
 # FIXME: Replace `main' with a function in `-ldl':
 
-AC_CHECK_LIB([nfnetlink], [nfnl_query] ,,,[-lnfnetlink])
-AC_CHECK_LIB([netfilter_conntrack], [nfct_query] ,,,[-lnetfilter_conntrack])
-
 AC_CHECK_HEADERS(arpa/inet.h)
 dnl check for inet_pton
 AC_CHECK_FUNCS(inet_pton)
@@ -127,9 +110,5 @@ dnl                  debug/src/Makefile
 dnl                  extensions/Makefile
 dnl                  src/Makefile])
 
-CFLAGS="$CFLAGS $LIBNETFILTER_CONNTRACK_CFLAGS"
-
-AC_SUBST(LIBNETFILTER_CONNTRACK_LIBS)
-
 AC_CONFIG_FILES([Makefile src/Makefile include/Makefile extensions/Makefile])
 AC_OUTPUT
diff --git a/src/Makefile.am b/src/Makefile.am
index 0e9af2e..a85a56f 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -7,8 +7,7 @@ CLEANFILES = read_config_yy.c read_config_lex.c
 sbin_PROGRAMS = conntrack conntrackd
 
 conntrack_SOURCES = conntrack.c
-conntrack_LDADD = ../extensions/libct_proto_tcp.la ../extensions/libct_proto_udp.la ../extensions/libct_proto_udplite.la ../extensions/libct_proto_icmp.la ../extensions/libct_proto_icmpv6.la ../extensions/libct_proto_sctp.la ../extensions/libct_proto_dccp.la ../extensions/libct_proto_gre.la ../extensions/libct_proto_unknown.la
-conntrack_LDFLAGS = @LIBNETFILTER_CONNTRACK_LIBS@
+conntrack_LDADD = ../extensions/libct_proto_tcp.la ../extensions/libct_proto_udp.la ../extensions/libct_proto_udplite.la ../extensions/libct_proto_icmp.la ../extensions/libct_proto_icmpv6.la ../extensions/libct_proto_sctp.la ../extensions/libct_proto_dccp.la ../extensions/libct_proto_gre.la ../extensions/libct_proto_unknown.la ${LIBNETFILTER_CONNTRACK_LIBS}
 
 conntrackd_SOURCES = alarm.c main.c run.c hash.c queue.c rbtree.c \
 		    local.c log.c mcast.c udp.c netlink.c vector.c \
@@ -28,6 +27,6 @@ conntrackd_SOURCES = alarm.c main.c run.c hash.c queue.c rbtree.c \
 # yacc and lex generate dirty code
 read_config_yy.o read_config_lex.o: AM_CFLAGS += -Wno-missing-prototypes -Wno-missing-declarations -Wno-implicit-function-declaration -Wno-nested-externs -Wno-undef -Wno-redundant-decls
 
-conntrackd_LDFLAGS = @LIBNETFILTER_CONNTRACK_LIBS@
+conntrackd_LDADD = ${LIBNETFILTER_CONNTRACK_LIBS}
 
 EXTRA_DIST = read_config_yy.h
-- 
1.7.1


^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 7/7] Add .gitignore files
  2010-10-30 22:49 conntrack-tools: build system cleanups Jan Engelhardt
                   ` (5 preceding siblings ...)
  2010-10-30 22:49 ` [PATCH 6/7] build: no need for error message in PKG_CHECK_MODULES Jan Engelhardt
@ 2010-10-30 22:49 ` Jan Engelhardt
  2010-11-01 21:37 ` conntrack-tools: build system cleanups Pablo Neira Ayuso
  7 siblings, 0 replies; 12+ messages in thread
From: Jan Engelhardt @ 2010-10-30 22:49 UTC (permalink / raw)
  To: pablo; +Cc: netfilter-devel

Signed-off-by: Jan Engelhardt <jengelh@medozas.de>
---
 .gitignore     |   18 ++++++++++++++++++
 src/.gitignore |    6 ++++++
 2 files changed, 24 insertions(+), 0 deletions(-)
 create mode 100644 .gitignore
 create mode 100644 src/.gitignore

diff --git a/.gitignore b/.gitignore
new file mode 100644
index 0000000..444c4b3
--- /dev/null
+++ b/.gitignore
@@ -0,0 +1,18 @@
+.deps
+.libs
+Makefile
+Makefile.in
+*.o
+*.la
+*.lo
+
+/aclocal.m4
+/autom4te.cache
+/config.*
+/configure
+/depcomp
+/install-sh
+/libtool
+/ltmain.sh
+/missing
+/ylwrap
diff --git a/src/.gitignore b/src/.gitignore
new file mode 100644
index 0000000..6e6763d
--- /dev/null
+++ b/src/.gitignore
@@ -0,0 +1,6 @@
+/conntrack
+/conntrackd
+
+/read_config_lex.c
+/read_config_yy.c
+/read_config_yy.h
-- 
1.7.1


^ permalink raw reply related	[flat|nested] 12+ messages in thread

* Re: conntrack-tools: build system cleanups
  2010-10-30 22:49 conntrack-tools: build system cleanups Jan Engelhardt
                   ` (6 preceding siblings ...)
  2010-10-30 22:49 ` [PATCH 7/7] Add .gitignore files Jan Engelhardt
@ 2010-11-01 21:37 ` Pablo Neira Ayuso
  2010-11-01 21:38   ` Jan Engelhardt
  7 siblings, 1 reply; 12+ messages in thread
From: Pablo Neira Ayuso @ 2010-11-01 21:37 UTC (permalink / raw)
  To: Jan Engelhardt; +Cc: netfilter-devel

On 31/10/10 00:49, Jan Engelhardt wrote:
> The following changes since commit 31ae56a2af1381459e4ec38c31ab810ffa410cf9:
> 
>   conntrack: allow to listen to all kind of expectation events (2010-10-12 12:56:01 +0200)
> 
> are available in the git repository at:
>   git://dev.medozas.de/conntrack-tools master

pulled and pushed this also. Thanks for all this build updates Jan :-).

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: conntrack-tools: build system cleanups
  2010-11-01 21:37 ` conntrack-tools: build system cleanups Pablo Neira Ayuso
@ 2010-11-01 21:38   ` Jan Engelhardt
  2010-11-01 21:40     ` Jan Engelhardt
  0 siblings, 1 reply; 12+ messages in thread
From: Jan Engelhardt @ 2010-11-01 21:38 UTC (permalink / raw)
  To: Pablo Neira Ayuso; +Cc: netfilter-devel

On Monday 2010-11-01 22:37, Pablo Neira Ayuso wrote:

>On 31/10/10 00:49, Jan Engelhardt wrote:
>> The following changes since commit 31ae56a2af1381459e4ec38c31ab810ffa410cf9:
>> 
>>   conntrack: allow to listen to all kind of expectation events (2010-10-12 12:56:01 +0200)
>> 
>> are available in the git repository at:
>>   git://dev.medozas.de/conntrack-tools master
>
>pulled and pushed this also. Thanks for all this build updates Jan :-).

Oh there's some more that have turned up, but I'm still piling. 
(Preparing a pull-req does take a static time.)

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: conntrack-tools: build system cleanups
  2010-11-01 21:38   ` Jan Engelhardt
@ 2010-11-01 21:40     ` Jan Engelhardt
  2010-11-01 22:09       ` Pablo Neira Ayuso
  0 siblings, 1 reply; 12+ messages in thread
From: Jan Engelhardt @ 2010-11-01 21:40 UTC (permalink / raw)
  To: Pablo Neira Ayuso; +Cc: netfilter-devel

On Monday 2010-11-01 22:38, Jan Engelhardt wrote:

>On Monday 2010-11-01 22:37, Pablo Neira Ayuso wrote:
>
>>On 31/10/10 00:49, Jan Engelhardt wrote:
>>> The following changes since commit 31ae56a2af1381459e4ec38c31ab810ffa410cf9:
>>> 
>>>   conntrack: allow to listen to all kind of expectation events (2010-10-12 12:56:01 +0200)
>>> 
>>> are available in the git repository at:
>>>   git://dev.medozas.de/conntrack-tools master
>>
>>pulled and pushed this also. Thanks for all this build updates Jan :-).

Speaking of which, what's Mr. Undocumented Commit 
178b917b75bc531843e45da0b06c05a73cba0ff2 doing in 
libnetfilter_conntrack?

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: conntrack-tools: build system cleanups
  2010-11-01 21:40     ` Jan Engelhardt
@ 2010-11-01 22:09       ` Pablo Neira Ayuso
  0 siblings, 0 replies; 12+ messages in thread
From: Pablo Neira Ayuso @ 2010-11-01 22:09 UTC (permalink / raw)
  To: Jan Engelhardt; +Cc: netfilter-devel

On 01/11/10 22:40, Jan Engelhardt wrote:
> On Monday 2010-11-01 22:38, Jan Engelhardt wrote:
> 
>> On Monday 2010-11-01 22:37, Pablo Neira Ayuso wrote:
>>
>>> On 31/10/10 00:49, Jan Engelhardt wrote:
>>>> The following changes since commit 31ae56a2af1381459e4ec38c31ab810ffa410cf9:
>>>>
>>>>   conntrack: allow to listen to all kind of expectation events (2010-10-12 12:56:01 +0200)
>>>>
>>>> are available in the git repository at:
>>>>   git://dev.medozas.de/conntrack-tools master
>>>
>>> pulled and pushed this also. Thanks for all this build updates Jan :-).
> 
> Speaking of which, what's Mr. Undocumented Commit 
> 178b917b75bc531843e45da0b06c05a73cba0ff2 doing in 
> libnetfilter_conntrack?

oops, that has slipped through, it was a unfinished patch in my stgit
stack. I have removed and forced a push. Sorry.

^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2010-11-01 22:09 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-10-30 22:49 conntrack-tools: build system cleanups Jan Engelhardt
2010-10-30 22:49 ` [PATCH 1/7] build: use autoconf-suggested naming of files Jan Engelhardt
2010-10-30 22:49 ` [PATCH 2/7] build: use modern call syntax for AM_INIT_AUTOMAKE Jan Engelhardt
2010-10-30 22:49 ` [PATCH 3/7] build: drop unused $(all_includes) Jan Engelhardt
2010-10-30 22:49 ` [PATCH 4/7] build: remove statements without effect Jan Engelhardt
2010-10-30 22:49 ` [PATCH 5/7] build: remove unused $(all_libraries) Jan Engelhardt
2010-10-30 22:49 ` [PATCH 6/7] build: no need for error message in PKG_CHECK_MODULES Jan Engelhardt
2010-10-30 22:49 ` [PATCH 7/7] Add .gitignore files Jan Engelhardt
2010-11-01 21:37 ` conntrack-tools: build system cleanups Pablo Neira Ayuso
2010-11-01 21:38   ` Jan Engelhardt
2010-11-01 21:40     ` Jan Engelhardt
2010-11-01 22:09       ` Pablo Neira Ayuso

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).