Skip to contentSkip to conversation

tcpdump: fix static build

PATCH New Baruch Siach 2 participants 2 messages Nov 11, 2014

lore
Share & tools
lore

Baruch Siach

author
#0 lore raw

PATCH 1/1 tcpdump: fix static build

Commit 746116d1eb2e (tcpdump: use libpcap shared library) broke static build of tcpdump, because its configure script doesn't take into account indirect dependencies of libpcap. Add these dependencies to the LIBS configure parameter.

Fixes:
http://autobuild.buildroot.net/results/fd231d34e4bb0306609b021e9e74eb862b7bd6bd/

Signed-off-by Baruch Siach <[email protected]>
---
 package/tcpdump/tcpdump.mk | 4 ++++
 1 file changed, 4 insertions(+)
package/tcpdump/tcpdump.mk +4 -0
index e62dfaf44a89..1acf7d9e3065 100644
@@ -16,6 +16,10 @@ TCPDUMP_DEPENDENCIES = zlib libpcap
 # Patching aclocal.m4
 TCPDUMP_AUTORECONF = YES
 
+ifeq ($(BR2_PREFER_STATIC_LIB),y)
+TCPDUMP_CONF_OPTS += LIBS="$(shell $(STAGING_DIR)/usr/bin/pcap-config --static --additional-libs)"
+endif
+
 # make install installs an unneeded extra copy of the tcpdump binary
 define TCPDUMP_REMOVE_DUPLICATED_BINARY
 	rm -f $(TARGET_DIR)/usr/sbin/tcpdump.$(TCPDUMP_VERSION)
Signature · 1 line
2.1.1
#1 lore raw

PATCH 1/1 tcpdump: fix static build

Quoted 11 lines
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line

"Baruch" == Baruch Siach <[email protected]> writes:

Commit 746116d1eb2e (tcpdump: use libpcap shared library) broke static build of tcpdump, because its configure script doesn't take into account indirect dependencies of libpcap. Add these dependencies to the LIBS configure parameter.

Fixes:
http://autobuild.buildroot.net/results/fd231d34e4bb0306609b021e9e74eb862b7bd6bd/

Signed-off-by Baruch Siach <[email protected]>

Committed, thanks.

Signature · 1 line
Bye, Peter Korsgaard