diff mbox

[nft] src: restore nft --debug and add missing \

Message ID 1416872298-6125-1-git-send-email-pablo@netfilter.org
State Not Applicable
Delegated to: Pablo Neira
Headers show

Commit Message

Pablo Neira Ayuso Nov. 24, 2014, 11:38 p.m. UTC
Add -DDEBUG to enable --debug option by default as it used to be before
the autotools conversion.

The missing \ at the end of the line causes LIBMNL_CFLAGS and
LIBNFTNL_CFLAGS to be ignored. This causes build failure if the libmnl
or libnftnl headers are not in a path that's already searched by the C
compiler. Reported by David Kozub.

Fixes: 5fa8e49 ("build: autotools conversion")
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
---
@Daniel: Your patch clashes with mine:
http://patchwork.ozlabs.org/patch/413457/

So I have merged your change to mine. Please, feel free to post your
Acked-by / Signed-off-by and I'll append it to this patch. Thanks.

 src/Makefile.am |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff mbox

Patch

diff --git a/src/Makefile.am b/src/Makefile.am
index 0a67810..d53c347 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -3,7 +3,7 @@  sbin_PROGRAMS = nft
 CLEANFILES = scanner.c parser_bison.c
 
 AM_CPPFLAGS = -I$(top_srcdir)/include
-AM_CPPFLAGS += -DDEFAULT_INCLUDE_PATH="\"${sysconfdir}\""
+AM_CPPFLAGS += -DDEFAULT_INCLUDE_PATH="\"${sysconfdir}\"" -DDEBUG \
 		${LIBMNL_CFLAGS} ${LIBNFTNL_CFLAGS}
 
 AM_CFLAGS = -Wall								\