From: Luca Boccassi <luca.boccassi@gmail.com>
Date: Fri, 1 Sep 2017 16:15:16 +0000 (+0100)
Subject: Backport patch to build DPDK with pkg-config
X-Git-Tag: collectd-5.7.2-2~11^2^2
X-Git-Url: https://git.tokkee.org/?a=commitdiff_plain;h=e2af5fddc3d85315147134e3b0a4d659d01ad433;p=pkg-collectd.git

Backport patch to build DPDK with pkg-config

Backport and adapt patches from upstream to use pkg-config when
querying for DPDK:

https://github.com/collectd/collectd/pull/2400
https://github.com/collectd/collectd/pull/2405

This allows DPDK in Debian to fix an upstream multi-arch issue,
where arch-dependents headers are installed in /usr/include breaking
multi-arch co-installability of libdpdk-dev.
The arch-dependent DPDK headers will be moved under /usr/include/<arch>
to fix the issue, and pkg-config --cflags will return the correct
-I values.

Closes #872482
---

diff --git a/debian/patches/dpdk-pkg-config.patch b/debian/patches/dpdk-pkg-config.patch
new file mode 100644
index 0000000..b69b19e
--- /dev/null
+++ b/debian/patches/dpdk-pkg-config.patch
@@ -0,0 +1,116 @@
+Description: configure.ac: dpdk: use pkg-config
+ To detect cflags and libs use the sometimes provided pkg-config for
+ libdpdk. That avoids build errors on systems where special flags are
+ needed and provided by dpdk via pkg-config, but not yet considered by
+ the collectd build system.
+Author: Christian Ehrhardt <christian.ehrhardt@canonical.com>
+Forwarded: https://github.com/collectd/collectd/pull/2400
+Reviewed-by: Luca Boccassi <bluca@debian.org>
+--- a/configure.ac
++++ b/configure.ac
+@@ -2607,34 +2607,69 @@ fi
+ 
+ # --with-libdpdk {{{
+ AC_ARG_VAR([LIBDPDK_CPPFLAGS], [Preprocessor flags for libdpdk])
++AC_ARG_VAR([LIBDPDK_CFLAGS], [Compiler flags for libdpdk])
+ AC_ARG_VAR([LIBDPDK_LDFLAGS], [Linker flags for libdpdk])
++AC_ARG_VAR([LIBDPDK_LIBS], [Libraries to link for libdpdk])
+ 
+-AC_ARG_WITH([libdpdk], [AS_HELP_STRING([--without-libdpdk], [Disable libdpdk.])])
++AC_ARG_WITH([libdpdk],
++  [AS_HELP_STRING([--without-libdpdk], [Disable libdpdk.])],
++  [with_libdpdk="$withval"],
++  [with_libdpdk="yes"]
++)
+ 
+-if test "x$with_libdpdk" != "xno"
+-then
+-	if test "x$LIBDPDK_CPPFLAGS" = "x"
+-	then
+-		LIBDPDK_CPPFLAGS="-I/usr/include/dpdk"
+-	fi
+-	SAVE_CPPFLAGS="$CPPFLAGS"
+-	CPPFLAGS="$LIBDPDK_CPPFLAGS $CPPFLAGS"
+-	AC_CHECK_HEADERS([rte_config.h],
+-		[with_libdpdk="yes"],
+-		[with_libdpdk="no (rte_config.h not found)"]
+-	)
+-	CPPFLAGS="$SAVE_CPPFLAGS"
++if test "x$with_libdpdk" != "xno"; then
++  PKG_CHECK_MODULES([DPDK], [libdpdk], [],
++		    [AC_MSG_NOTICE([no DPDK pkg-config, using defaults])])
++  if test "x$LIBDPDK_CPPFLAGS" = "x"; then
++    LIBDPDK_CPPFLAGS="-I/usr/include/dpdk"
++  fi
++  if test "x$LIBDPDK_CFLAGS" = "x"; then
++      LIBDPDK_CFLAGS="$DPDK_CFLAGS"
++      LIBDPDK_CPPFLAGS="$LIBDPDK_CPPFLAGS $DPDK_CFLAGS"
++  fi
++  if test "x$LIBDPDK_LIBS" = "x"; then
++      if test "x$DPDK_LIBS" != "x"; then
++          LIBDPDK_LIBS="$DPDK_LIBS"
++      else
++          LIBDPDK_LIBS="-ldpdk"
++      fi
++  fi
++  SAVE_CPPFLAGS="$CPPFLAGS"
++  CPPFLAGS="$LIBDPDK_CPPFLAGS $CPPFLAGS"
++  SAVE_CFLAGS="$CFLAGS"
++  CFLAGS="$LIBDPDK_CFLAGS $CFLAGS"
++  AC_CHECK_HEADERS([rte_config.h],
++    [
++      with_libdpdk="yes"
++      AC_PREPROC_IFELSE(
++        [
++          AC_LANG_SOURCE(
++            [[
++              #include <rte_version.h>
++              #if RTE_VERSION < RTE_VERSION_NUM(16,7,0,0)
++              #error "required DPDK >= 16.07"
++              #endif
++            ]]
++          )
++        ],
++        [dpdk_keepalive="yes"],
++        [dpdk_keepalive="no (DPDK version < 16.07)"]
++      )
++    ],
++    [with_libdpdk="no (rte_config.h not found)"]
++  )
++  CPPFLAGS="$SAVE_CPPFLAGS"
++  CFLAGS="$SAVE_CFLAGS"
+ fi
+ 
+-if test "x$with_libdpdk" = "xyes"
+-then
+-	SAVE_LDFLAGS="$LDFLAGS"
+-	LDFLAGS="$LIBDPDK_LDFLAGS $LDFLAGS"
+-	AC_CHECK_LIB([dpdk], [rte_eal_init],
+-		[with_libdpdk="yes"],
+-		[with_libdpdk="no (symbol 'rte_eal_init' not found)"]
+-	)
+-	LDFLAGS="$SAVE_LDFLAGS"
++if test "x$with_libdpdk" = "xyes"; then
++  SAVE_LDFLAGS="$LDFLAGS"
++  LDFLAGS="$LIBDPDK_LDFLAGS $LDFLAGS"
++  AC_CHECK_LIB([dpdk], [rte_eal_init],
++    [with_libdpdk="yes"],
++    [with_libdpdk="no (symbol 'rte_eal_init' not found)"]
++  )
++  LDFLAGS="$SAVE_LDFLAGS"
+ fi
+ 
+ # }}}
+--- a/src/Makefile.am
++++ b/src/Makefile.am
+@@ -420,8 +420,9 @@ if BUILD_PLUGIN_DPDKSTAT
+ pkglib_LTLIBRARIES += dpdkstat.la
+ dpdkstat_la_SOURCES = dpdkstat.c
+ dpdkstat_la_CPPFLAGS = $(AM_CPPFLAGS) $(LIBDPDK_CPPFLAGS)
++dpdkstat_la_CFLAGS = $(AM_CFLAGS) $(LIBDPDK_CFLAGS)
+ dpdkstat_la_LDFLAGS = $(PLUGIN_LDFLAGS) $(LIBDPDK_LDFLAGS)
+-dpdkstat_la_LIBADD = -ldpdk
++dpdkstat_la_LIBADD = $(LIBDPDK_LIBS)
+ endif
+ 
+ if BUILD_PLUGIN_DRBD
diff --git a/debian/patches/series b/debian/patches/series
index ecd9ee9..5630e99 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -4,3 +4,4 @@ myplugin_includes.patch
 nagios-debian-paths.patch
 libcollectdclient_error_buffer.patch
 local-msr-index-h.patch
+dpdk-pkg-config.patch