forked from osmocom/wireshark
autotools: Use explicit zlib dependency build flags
Use pkg-config if a zlib.pc file is available. Remove the now redundant AC_TRY_LINK_FUNC test (there are no linker flags for GTK+ here). Change-Id: I7de744749eba7231ae0097b975144b76ffcf1bdb Reviewed-on: https://code.wireshark.org/review/14263 Petri-Dish: João Valverde <j@v6e.pt> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: João Valverde <j@v6e.pt>
This commit is contained in:
parent
4fca153089
commit
2bded0b661
24
Makefile.am
24
Makefile.am
|
@ -518,7 +518,8 @@ text2pcap_CFLAGS = $(GENERATED_CFLAGS)
|
|||
# Libraries with which to link text2pcap.
|
||||
text2pcap_LDADD = \
|
||||
wsutil/libwsutil.la \
|
||||
@GLIB_LIBS@
|
||||
@GLIB_LIBS@ \
|
||||
@ZLIB_LIBS@
|
||||
|
||||
mergecap_CPPFLAGS = $(AM_CPPFLAGS) $(GLIB_CFLAGS)
|
||||
|
||||
|
@ -526,7 +527,8 @@ mergecap_CPPFLAGS = $(AM_CPPFLAGS) $(GLIB_CFLAGS)
|
|||
mergecap_LDADD = \
|
||||
wiretap/libwiretap.la \
|
||||
wsutil/libwsutil.la \
|
||||
@GLIB_LIBS@
|
||||
@GLIB_LIBS@ \
|
||||
@ZLIB_LIBS@
|
||||
|
||||
capinfos_CPPFLAGS = $(AM_CPPFLAGS) $(GLIB_CFLAGS)
|
||||
|
||||
|
@ -535,7 +537,8 @@ capinfos_LDADD = \
|
|||
wiretap/libwiretap.la \
|
||||
wsutil/libwsutil.la \
|
||||
@GLIB_LIBS@ \
|
||||
@LIBGCRYPT_LIBS@
|
||||
@LIBGCRYPT_LIBS@ \
|
||||
@ZLIB_LIBS@
|
||||
|
||||
captype_CPPFLAGS = $(AM_CPPFLAGS) $(GLIB_CFLAGS)
|
||||
|
||||
|
@ -543,7 +546,8 @@ captype_CPPFLAGS = $(AM_CPPFLAGS) $(GLIB_CFLAGS)
|
|||
captype_LDADD = \
|
||||
wiretap/libwiretap.la \
|
||||
wsutil/libwsutil.la \
|
||||
@GLIB_LIBS@
|
||||
@GLIB_LIBS@ \
|
||||
@ZLIB_LIBS@
|
||||
|
||||
editcap_CPPFLAGS = $(AM_CPPFLAGS) $(GLIB_CFLAGS)
|
||||
|
||||
|
@ -551,7 +555,8 @@ editcap_CPPFLAGS = $(AM_CPPFLAGS) $(GLIB_CFLAGS)
|
|||
editcap_LDADD = \
|
||||
wiretap/libwiretap.la \
|
||||
wsutil/libwsutil.la \
|
||||
@GLIB_LIBS@
|
||||
@GLIB_LIBS@ \
|
||||
@ZLIB_LIBS@
|
||||
|
||||
reordercap_CPPFLAGS = $(AM_CPPFLAGS) $(GLIB_CFLAGS)
|
||||
|
||||
|
@ -559,7 +564,8 @@ reordercap_CPPFLAGS = $(AM_CPPFLAGS) $(GLIB_CFLAGS)
|
|||
reordercap_LDADD = \
|
||||
wiretap/libwiretap.la \
|
||||
wsutil/libwsutil.la \
|
||||
@GLIB_LIBS@
|
||||
@GLIB_LIBS@ \
|
||||
@ZLIB_LIBS@
|
||||
|
||||
randpkt_CPPFLAGS = $(AM_CPPFLAGS) $(GLIB_CFLAGS)
|
||||
|
||||
|
@ -573,7 +579,8 @@ randpkt_LDADD = \
|
|||
@SOCKET_LIBS@ \
|
||||
@NSL_LIBS@ \
|
||||
@C_ARES_LIBS@ \
|
||||
@ADNS_LIBS@
|
||||
@ADNS_LIBS@ \
|
||||
@ZLIB_LIBS@
|
||||
|
||||
dftest_CPPFLAGS = $(AM_CPPFLAGS) $(GLIB_CFLAGS)
|
||||
|
||||
|
@ -630,7 +637,8 @@ dumpcap_LDADD = \
|
|||
@SYSTEMCONFIGURATION_FRAMEWORKS@ \
|
||||
@COREFOUNDATION_FRAMEWORKS@ \
|
||||
@LIBCAP_LIBS@ \
|
||||
@LIBNL_LIBS@
|
||||
@LIBNL_LIBS@ \
|
||||
@ZLIB_LIBS@
|
||||
|
||||
#
|
||||
# Build the version string
|
||||
|
|
100
acinclude.m4
100
acinclude.m4
|
@ -593,31 +593,45 @@ AC_DEFUN([AC_WIRESHARK_PCAP_REMOTE_CHECK],
|
|||
AC_DEFUN([AC_WIRESHARK_ZLIB_CHECK],
|
||||
[
|
||||
AC_WIRESHARK_PUSH_FLAGS
|
||||
ac_ws_saved_LIBS="$LIBS"
|
||||
|
||||
if test "x$zlib_dir" != "x"
|
||||
if test "x$zlib_dir" = "x"
|
||||
then
|
||||
#
|
||||
# The user specified a directory in which zlib resides,
|
||||
# so add the "include" subdirectory of that directory to
|
||||
# the include file search path and the "lib" subdirectory
|
||||
# of that directory to the library search path.
|
||||
#
|
||||
# XXX - if there's also a zlib in a directory that's
|
||||
# already in CPPFLAGS or LDFLAGS, this won't make us find
|
||||
# the version in the specified directory, as the compiler
|
||||
# and/or linker will search that other directory before it
|
||||
# searches the specified directory.
|
||||
#
|
||||
CPPFLAGS="$CPPFLAGS -I$zlib_dir/include"
|
||||
AC_WIRESHARK_ADD_DASH_L(LDFLAGS, $zlib_dir/lib)
|
||||
PKG_WIRESHARK_CHECK_SYSTEM_MODULES(ZLIB, [zlib], [zlib_found=yes], [zlib_found=no])
|
||||
fi
|
||||
|
||||
if test x"$zlib_found" != xyes; then
|
||||
if test "x$zlib_dir" != "x"; then
|
||||
#
|
||||
# The user specified a directory in which zlib resides,
|
||||
# so add the "include" subdirectory of that directory to
|
||||
# the include file search path and the "lib" subdirectory
|
||||
# of that directory to the library search path.
|
||||
#
|
||||
ZLIB_CFLAGS="-I$zlib_dir/include"
|
||||
AC_WIRESHARK_ADD_DASH_L(ZLIB_LIBS, [$zlib_dir/lib])
|
||||
fi
|
||||
AX_APPEND_FLAG(-lz, ZLIB_LIBS)
|
||||
fi
|
||||
|
||||
#
|
||||
# Make sure we have "zlib.h". If we don't, it means we probably
|
||||
# don't have zlib, so don't use it.
|
||||
# Make sure we have "zlib.h".
|
||||
#
|
||||
AC_CHECK_HEADER(zlib.h,,
|
||||
CPPFLAGS="$ZLIB_CFLAGS $CPPFLAGS"
|
||||
AC_CHECK_HEADERS(zlib.h,
|
||||
[
|
||||
#
|
||||
# We link with zlib to support uncompression of
|
||||
# gzipped network traffic, e.g. in an HTTP request
|
||||
# or response body.
|
||||
#
|
||||
have_zlib=yes
|
||||
],
|
||||
[
|
||||
have_zlib=no
|
||||
ZLIB_CFLAGS=""
|
||||
ZLIB_LIBS=""
|
||||
|
||||
if test "x$zlib_dir" != "x"
|
||||
then
|
||||
#
|
||||
|
@ -641,58 +655,24 @@ AC_DEFUN([AC_WIRESHARK_ZLIB_CHECK],
|
|||
# We couldn't find the header file; don't use the
|
||||
# library, as it's probably not present.
|
||||
#
|
||||
want_zlib=no
|
||||
AC_MSG_NOTICE(zlib.h not found - disabling gzip compression and decompression)
|
||||
fi
|
||||
fi
|
||||
])
|
||||
|
||||
if test "x$want_zlib" != "xno"
|
||||
if test "x$have_zlib" != "xno"
|
||||
then
|
||||
#
|
||||
# Well, we at least have the zlib header file.
|
||||
# We link with zlib to support uncompression of
|
||||
# gzipped network traffic, e.g. in an HTTP request
|
||||
# or response body.
|
||||
#
|
||||
if test "x$zlib_dir" != "x"
|
||||
then
|
||||
WS_CPPFLAGS="$WS_CPPFLAGS -I$zlib_dir/include"
|
||||
AC_WIRESHARK_ADD_DASH_L(WS_LDFLAGS, $zlib_dir/lib)
|
||||
fi
|
||||
LIBS="$LIBS -lz"
|
||||
AC_DEFINE(HAVE_LIBZ, 1, [Define to use libz library])
|
||||
#
|
||||
# Check for "inflatePrime()" in zlib, which we need
|
||||
# in order to read compressed capture files.
|
||||
#
|
||||
AC_CHECK_FUNCS(inflatePrime)
|
||||
|
||||
if test "x$ac_cv_func_inflatePrime" = "xyes" ; then
|
||||
#
|
||||
# Now check for "inflatePrime()" in zlib when
|
||||
# linking with the linker flags for GTK+
|
||||
# applications; people often grab XFree86 source
|
||||
# and build and install it on their systems,
|
||||
# and they appear sometimes to misconfigure
|
||||
# XFree86 so that, even on systems with zlib,
|
||||
# it assumes there is no zlib, so the XFree86
|
||||
# build process builds and installs its
|
||||
# own zlib in the X11 library directory.
|
||||
#
|
||||
# The zlib in at least some versions of XFree86
|
||||
# is an older version that may lack "inflatePrime()",
|
||||
# and that's the zlib with which Wireshark gets
|
||||
# linked, so the build of Wireshark fails.
|
||||
#
|
||||
AC_MSG_CHECKING([for inflatePrime missing when linking with X11])
|
||||
AC_TRY_LINK_FUNC(inflatePrime, AC_MSG_RESULT(no),
|
||||
[
|
||||
AC_MSG_RESULT(yes)
|
||||
AC_MSG_ERROR(old zlib found when linking with X11 - get rid of old zlib.)
|
||||
])
|
||||
fi
|
||||
LIBS="$ZLIB_LIBS $LIBS"
|
||||
AC_CHECK_FUNCS(inflatePrime,[],
|
||||
[AC_MSG_NOTICE(inflatePrime not found - disabling gzipped capture file support)])
|
||||
fi
|
||||
|
||||
AC_SUBST(ZLIB_CFLAGS)
|
||||
AC_SUBST(ZLIB_LIBS)
|
||||
LIBS="$ac_ws_saved_LIBS"
|
||||
AC_WIRESHARK_POP_FLAGS
|
||||
])
|
||||
|
||||
|
|
24
configure.ac
24
configure.ac
|
@ -2424,8 +2424,11 @@ else
|
|||
AC_WIRESHARK_PCAP_REMOTE_CHECK
|
||||
fi
|
||||
|
||||
dnl
|
||||
dnl zlib check
|
||||
dnl
|
||||
AC_MSG_CHECKING(whether to use zlib for gzip compression and decompression)
|
||||
have_zlib=no
|
||||
|
||||
AC_ARG_WITH(zlib,
|
||||
AC_HELP_STRING([--with-zlib@<:@=DIR@:>@],
|
||||
|
@ -2442,24 +2445,11 @@ AC_ARG_WITH(zlib,
|
|||
zlib_dir="$withval"
|
||||
fi
|
||||
],[
|
||||
#
|
||||
# Use zlib if it's present, otherwise don't.
|
||||
#
|
||||
want_zlib=ifavailable
|
||||
zlib_dir=
|
||||
want_zlib="if available"
|
||||
])
|
||||
if test "x$want_zlib" = "xno" ; then
|
||||
AC_MSG_RESULT(no)
|
||||
else
|
||||
AC_MSG_RESULT(yes)
|
||||
AC_MSG_RESULT([$want_zlib])
|
||||
if test "x$want_zlib" != "xno" ; then
|
||||
AC_WIRESHARK_ZLIB_CHECK
|
||||
if test "x$want_zlib" = "xno" ; then
|
||||
AC_MSG_RESULT(zlib not found - disabling gzip compression and decompression)
|
||||
else
|
||||
if test "x$ac_cv_func_inflatePrime" = "xno" ; then
|
||||
AC_MSG_RESULT(inflatePrime not found in zlib - disabling gzipped capture file support)
|
||||
fi
|
||||
fi
|
||||
fi
|
||||
|
||||
dnl Lua check
|
||||
|
@ -3370,7 +3360,7 @@ else
|
|||
dumpcap_group_message="$DUMPCAP_GROUP"
|
||||
fi
|
||||
|
||||
if test "x$want_zlib" = "xno" ; then
|
||||
if test "x$have_zlib" = "xno" ; then
|
||||
zlib_message="no"
|
||||
else
|
||||
zlib_message="yes"
|
||||
|
|
|
@ -43,7 +43,7 @@ ACLOCAL_AMFLAGS = `../aclocal-flags`
|
|||
AM_CPPFLAGS = $(INCLUDEDIRS) -I$(builddir)/wslua $(WS_CPPFLAGS) \
|
||||
$(GLIB_CFLAGS) $(LUA_CFLAGS) $(LIBGNUTLS_CFLAGS) \
|
||||
$(LIBGCRYPT_CFLAGS) $(LIBSMI_CFLAGS) $(LIBGEOIP_CFLAGS) \
|
||||
$(KRB5_CFLAGS)
|
||||
$(KRB5_CFLAGS) $(ZLIB_CFLAGS)
|
||||
|
||||
noinst_LTLIBRARIES = libwireshark_generated.la libwireshark_asmopt.la
|
||||
lib_LTLIBRARIES = libwireshark.la
|
||||
|
@ -147,6 +147,7 @@ libwireshark_la_LIBADD = \
|
|||
@LIBSMI_LDFLAGS@ \
|
||||
@GEOIP_LIBS@ \
|
||||
@GLIB_LIBS@ \
|
||||
$(ZLIB_LIBS) \
|
||||
-lm
|
||||
|
||||
libwireshark_la_DEPENDENCIES = \
|
||||
|
@ -168,17 +169,17 @@ EXTRA_PROGRAMS = reassemble_test tvbtest oids_test
|
|||
reassemble_test_LDADD = \
|
||||
libwireshark.la \
|
||||
$(GLIB_LIBS) \
|
||||
-lz
|
||||
$(ZLIB_LIBS)
|
||||
|
||||
tvbtest_LDADD = \
|
||||
libwireshark.la \
|
||||
$(GLIB_LIBS) \
|
||||
-lz
|
||||
$(ZLIB_LIBS)
|
||||
|
||||
oids_test_LDADD = \
|
||||
libwireshark.la \
|
||||
$(GLIB_LIBS) \
|
||||
-lz
|
||||
$(ZLIB_LIBS)
|
||||
|
||||
exntest: exntest.o except.o
|
||||
$(LINK) $^ $(GLIB_LIBS)
|
||||
|
|
|
@ -22,7 +22,8 @@ include Makefile.common
|
|||
include $(top_srcdir)/Makefile.am.inc
|
||||
|
||||
AM_CPPFLAGS = $(INCLUDEDIRS) -I$(top_srcdir)/epan $(WS_CPPFLAGS) \
|
||||
$(GLIB_CFLAGS) $(LIBGNUTLS_CFLAGS) $(LIBGCRYPT_CFLAGS)
|
||||
$(GLIB_CFLAGS) $(LIBGNUTLS_CFLAGS) $(LIBGCRYPT_CFLAGS) \
|
||||
$(ZLIB_CFLAGS)
|
||||
|
||||
noinst_LTLIBRARIES = libdirtydissectors.la libfiledissectors.la libdissectors.la
|
||||
|
||||
|
|
|
@ -22,7 +22,7 @@
|
|||
include $(top_srcdir)/Makefile.am.inc
|
||||
|
||||
AM_CPPFLAGS = $(INCLUDEDIRS) $(WS_CPPFLAGS) $(LUA_CFLAGS) \
|
||||
$(GLIB_CFLAGS)
|
||||
$(GLIB_CFLAGS) $(ZLIB_CFLAGS)
|
||||
|
||||
noinst_LTLIBRARIES = libwslua.la
|
||||
|
||||
|
|
|
@ -24,7 +24,8 @@ include $(top_srcdir)/Makefile.am.inc
|
|||
|
||||
ACLOCAL_AMFLAGS = `../aclocal-flags`
|
||||
|
||||
AM_CPPFLAGS = $(INCLUDEDIRS) $(WS_CPPFLAGS) -DWS_BUILD_DLL $(GLIB_CFLAGS)
|
||||
AM_CPPFLAGS = $(INCLUDEDIRS) $(WS_CPPFLAGS) -DWS_BUILD_DLL $(GLIB_CFLAGS) \
|
||||
$(ZLIB_CFLAGS)
|
||||
|
||||
noinst_LTLIBRARIES = libwiretap_generated.la
|
||||
lib_LTLIBRARIES = libwiretap.la
|
||||
|
@ -63,7 +64,11 @@ EXTRA_DIST = \
|
|||
$(GENERATOR_FILES) \
|
||||
$(GENERATED_FILES)
|
||||
|
||||
libwiretap_la_LIBADD = libwiretap_generated.la ${top_builddir}/wsutil/libwsutil.la $(GLIB_LIBS)
|
||||
libwiretap_la_LIBADD = \
|
||||
libwiretap_generated.la \
|
||||
${top_builddir}/wsutil/libwsutil.la \
|
||||
$(GLIB_LIBS) \
|
||||
$(ZLIB_LIBS)
|
||||
|
||||
libwiretap_la_DEPENDENCIES = libwiretap_generated.la ${top_builddir}/wsutil/libwsutil.la
|
||||
|
||||
|
|
Loading…
Reference in New Issue