From: Pablo Neira Ayuso <pablo@netfilter.org>
To: Hangbin Liu <liuhangbin@gmail.com>
Cc: netfilter dev <netfilter@vger.kernel.org>,
netfilter-devel@vger.kernel.org
Subject: Re: [contrack-tools PATCH] build: do not compile nfct if we disabled cttimeout
Date: Fri, 9 May 2014 15:59:42 +0200 [thread overview]
Message-ID: <20140509135942.GA11325@localhost> (raw)
In-Reply-To: <20140509115411.GJ25631@localhost.localdomain>
On Fri, May 09, 2014 at 07:54:11PM +0800, Hangbin Liu wrote:
> On Fri, May 09, 2014 at 01:29:59PM +0200, Pablo Neira Ayuso wrote:
> > On Thu, May 08, 2014 at 06:02:25PM +0800, Hangbin Liu wrote:
> > > If we do not have libnetfilter_cttimeout and disabled cttimeout, conntrack-tools
> > > will compile fail because nfct.c include nfnetlink_cttimeout.h. Fix it by only
> > > compile nfct when we have enabled cttimeout
> >
> > Strange. The nfnetlink_cttimeout.h file is cached inside the
> > conntrack-tools tree:
> >
> > conntrack-tools/include/linux/netfilter/nfnetlink_cttimeout.h
> >
> > Here this compiles fine with my current tree and using:
> >
> > ./configure --prefix=/usr --disable-cthelper --disable-cttimeout
> >
> > Please, investigate this issue a bit further.
>
> That's weird, on my system without libnetfilter_cttimeout.
> # ./configure --prefix=/usr
> checking build system type... x86_64-unknown-linux-gnu
> checking host system type... x86_64-unknown-linux-gnu
> checking for a BSD-compatible install... /usr/bin/install -c
> checking whether build environment is sane... yes
>
> <snip>
>
> checking for LIBNETFILTER_CONNTRACK... yes
> checking for LIBNETFILTER_CTTIMEOUT... no
> configure: error: Package requirements (libnetfilter_cttimeout >= 1.0.0) were not met:
>
> No package 'libnetfilter_cttimeout' found
>
> Consider adjusting the PKG_CONFIG_PATH environment variable if you
> installed software in a non-standard prefix.
>
> Alternatively, you may set the environment variables LIBNETFILTER_CTTIMEOUT_CFLAGS
> and LIBNETFILTER_CTTIMEOUT_LIBS to avoid the need to call pkg-config.
> See the pkg-config man page for more details.
>
> # ./configure --prefix=/usr --disable-cthelper --disable-cttimeout
>
> <snip>
>
> config.status: executing depfiles commands
> config.status: executing libtool commands
>
> conntrack-tools configuration:
> userspace conntrack helper support: no
> conntrack timeout support: no
>
> # make
> Making all in extensions
> make[1]: Entering directory `/root/conntrack-tools/extensions'
> CC libct_proto_tcp.lo
> CCLD libct_proto_tcp.la
>
> <snip>
>
> YACC read_config_yy.c
> updating read_config_yy.h
> CC read_config_yy.o
> LEX read_config_lex.c
> CC read_config_lex.o
> read_config_lex.c: In function ‘yylex’:
> read_config_lex.c:2333:23: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
> for ( yyl = 0; yyl < yyleng; ++yyl )
> ^
> read_config_lex.c: In function ‘yy_scan_bytes’:
> read_config_lex.c:3658:17: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
> for ( i = 0; i < _yybytes_len; ++i )
> ^
> CC stack.o
> CCLD conntrackd
> CC nfct.o
> nfct.c:26:59: fatal error: libnetfilter_cttimeout/libnetfilter_cttimeout.h: No such file or directory
> #include <libnetfilter_cttimeout/libnetfilter_cttimeout.h>
> ^
OK, I see. That's not nfnetlink_cttimeout.h, it's
libnetfilter_cttimeout.h.
Let me check if I can come up with some alternative solution.
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2014-05-09 13:59 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-05-08 10:02 [contrack-tools PATCH] build: do not compile nfct if we disabled cttimeout Hangbin Liu
2014-05-09 11:29 ` Pablo Neira Ayuso
2014-05-09 11:54 ` Hangbin Liu
2014-05-09 13:59 ` Pablo Neira Ayuso [this message]
2014-05-10 2:51 ` Hangbin Liu
2014-05-12 15:23 ` Pablo Neira Ayuso
2014-05-12 15:24 ` Pablo Neira Ayuso
2014-05-12 15:43 ` Hangbin Liu
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20140509135942.GA11325@localhost \
--to=pablo@netfilter.org \
--cc=liuhangbin@gmail.com \
--cc=netfilter-devel@vger.kernel.org \
--cc=netfilter@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.