netfilter-devel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] build: resolve link failure for ip6t_NETMAP
@ 2012-12-02 18:16 Jan Engelhardt
  2012-12-03 13:12 ` Pablo Neira Ayuso
  0 siblings, 1 reply; 14+ messages in thread
From: Jan Engelhardt @ 2012-12-02 18:16 UTC (permalink / raw)
  To: pablo; +Cc: netfilter-devel

Link stage of libip6t_NETMAP failed since recently.

  CCLD     libip6t_NETMAP.so
/usr/lib64/gcc/x86_64-suse-linux/4.7/../../../../x86_64-suse-linux/bin/ld:
cannot find -lip6tc

Signed-off-by: Jan Engelhardt <jengelh@inai.de>
---

Retrievable through
	git://git.inai.de/iptables master


 extensions/GNUmakefile.in |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/extensions/GNUmakefile.in b/extensions/GNUmakefile.in
index e71e3ff..68e0b9b 100644
--- a/extensions/GNUmakefile.in
+++ b/extensions/GNUmakefile.in
@@ -101,7 +101,8 @@ libxt_state.so: libxt_conntrack.so
 	ln -fs $< $@
 
 # Need the LIBADDs in iptables/Makefile.am too for libxtables_la_LIBADD
-ip6t_NETMAP_LIBADD  = -lip6tc
+ip6t_NETMAP_LIBADD  = -L${top_builddir}/libiptc/.libs \
+                      -L${top_builddir}/libiptc -lip6tc
 xt_RATEEST_LIBADD   = -lm
 xt_statistic_LIBADD = -lm
 
-- 
1.7.10.4


^ permalink raw reply related	[flat|nested] 14+ messages in thread
* Re: [PATCH] extensions: fix linking against -lip6tc
@ 2012-12-31  0:51 Mike Frysinger
  2012-12-31  5:54 ` [PATCH] build: resolve link failure for ip6t_NETMAP Jan Engelhardt
  0 siblings, 1 reply; 14+ messages in thread
From: Mike Frysinger @ 2012-12-31  0:51 UTC (permalink / raw)
  To: Pablo Neira Ayuso; +Cc: netfilter-devel

[-- Attachment #1: Type: Text/Plain, Size: 856 bytes --]

On Sunday 30 December 2012 18:38:28 Pablo Neira Ayuso wrote:
> Hi Mike,
> 
> On Sun, Dec 30, 2012 at 06:11:47PM -0500, Mike Frysinger wrote:
> > The current build forgets to specify a path to find libip6tc which means
> > it either fails (if there is no libip6tc in the system), or links against
> > an old version (if there is one in the system).
> 
> We already got a similar patch fix for that issue:
> 
> http://patchwork.ozlabs.org/patch/208133/
> 
> Can you give it a try and check if it also fixes the problem for you?

adding an -L path to the non-.libs/ subdir doesn't make sense when the 
existing code (the -lxtables stuff) only uses .libs/.

really the best thing would to stop grubbing around in libtool internals and 
use libtool to link considering it was used to generate the other libraries in 
the first place.
-mike

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

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

end of thread, other threads:[~2013-01-12  1:37 UTC | newest]

Thread overview: 14+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-12-02 18:16 [PATCH] build: resolve link failure for ip6t_NETMAP Jan Engelhardt
2012-12-03 13:12 ` Pablo Neira Ayuso
2012-12-03 15:58   ` Jan Engelhardt
2012-12-04 12:16     ` Pablo Neira Ayuso
2012-12-04 13:42       ` Jan Engelhardt
  -- strict thread matches above, loose matches on Subject: below --
2012-12-31  0:51 [PATCH] extensions: fix linking against -lip6tc Mike Frysinger
2012-12-31  5:54 ` [PATCH] build: resolve link failure for ip6t_NETMAP Jan Engelhardt
2013-01-02  4:03   ` Mike Frysinger
2013-01-02  8:42     ` Jan Engelhardt
2013-01-02  8:47       ` Jan Engelhardt
2013-01-02 17:23         ` Mike Frysinger
2013-01-02 23:50           ` Pablo Neira Ayuso
2013-01-11 23:34         ` Dmitry V. Levin
2013-01-12  0:00           ` Dmitry V. Levin
2013-01-12  1:37           ` Jan Engelhardt

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).