You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by zw...@apache.org on 2018/07/31 20:43:52 UTC
[trafficserver] 04/11: Cleans up brotli compiler and linker flags
This is an automated email from the ASF dual-hosted git repository.
zwoop pushed a commit to branch 8.0.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git
commit f84abcf1de2b54cef00d6a16b516d393e6d24d56
Author: Randall Meyer <ra...@yahoo.com>
AuthorDate: Thu Jul 12 12:35:36 2018 -0700
Cleans up brotli compiler and linker flags
Use more standard form of X_FLAGS variable naming and switch to
uppercase LIBBROTLIENC for pkg-config to match other libraries
(cherry picked from commit c3f0582e9273cc6cfea6ffd96a0c2432828f7110)
---
build/brotli.m4 | 18 +++++++++---------
plugins/compress/Makefile.inc | 4 ++--
2 files changed, 11 insertions(+), 11 deletions(-)
diff --git a/build/brotli.m4 b/build/brotli.m4
index 1984552..a3189b6 100644
--- a/build/brotli.m4
+++ b/build/brotli.m4
@@ -31,8 +31,8 @@ AC_ARG_WITH(brotli, [AC_HELP_STRING([--with-brotli=DIR],[use a specific brotli l
has_brotli=1
case "$withval" in
*":"*)
- brotli_include="`echo $withval |sed -e 's/:.*$//'`"
- brotli_ldflags="`echo $withval |sed -e 's/^.*://'`"
+ brotli_include="`echo $withval | sed -e 's/:.*$//'`"
+ brotli_ldflags="`echo $withval | sed -e 's/^.*://'`"
AC_MSG_CHECKING(checking for brotli includes in $brotli_include libs in $brotli_ldflags )
;;
*)
@@ -66,8 +66,8 @@ if test "$has_brotli" != "0"; then
AC_CHECK_HEADERS(brotli/encode.h, [brotli_have_headers=1])
fi
if test "$brotli_have_headers" != "0"; then
- AC_SUBST([LIB_BROTLIENC], [-lbrotlienc])
- AC_SUBST([CFLAGS_BROTLIENC], [-I${brotli_include}])
+ AC_SUBST([BROTLIENC_LIB], [-lbrotlienc])
+ AC_SUBST([BROTLIENC_CFLAGS], [-I${brotli_include}])
else
has_brotli=0
CPPFLAGS=$saved_cppflags
@@ -80,15 +80,15 @@ AC_CHECK_HEADER([brotli/encode.h], [], [has_brotli=0])
AC_CHECK_LIB([brotlienc], BrotliEncoderCreateInstance, [], [has_brotli=0])
if test "x$has_brotli" == "x0"; then
- PKG_CHECK_EXISTS([libbrotlienc],
+ PKG_CHECK_EXISTS([LIBBROTLIENC],
[
- PKG_CHECK_MODULES([libbrotlienc], [libbrotlienc >= 0.6.0], [
- AC_SUBST([LIB_BROTLIENC], [$libbrotlienc_LIBS])
- AC_SUBST([CFLAGS_BROTLIENC], [$libbrotlienc_CFLAGS])
+ PKG_CHECK_MODULES([LIBBROTLIENC], [libbrotlienc >= 0.6.0], [
+ AC_SUBST([BROTLIENC_LIB], [$LIBBROTLIENC_LIBS])
+ AC_SUBST([BROTLIENC_CFLAGS], [$LIBBROTLIENC_CFLAGS])
], [])
], [])
else
- AC_SUBST([LIB_BROTLIENC], [-lbrotlienc])
+ AC_SUBST([BROTLIENC_LIB], [-lbrotlienc])
fi
])
diff --git a/plugins/compress/Makefile.inc b/plugins/compress/Makefile.inc
index 223fd5d..329d174 100644
--- a/plugins/compress/Makefile.inc
+++ b/plugins/compress/Makefile.inc
@@ -18,6 +18,6 @@ pkglib_LTLIBRARIES += compress/compress.la
compress_compress_la_SOURCES = compress/compress.cc compress/configuration.cc compress/misc.cc
compress_compress_la_LDFLAGS = \
- $(AM_LDFLAGS) $(LIB_BROTLIENC)
+ $(AM_LDFLAGS) $(BROTLIENC_LIB)
-compress_compress_la_CXXFLAGS = $(AM_CXXFLAGS) $(CFLAGS_BROTLIENC)
+compress_compress_la_CXXFLAGS = $(AM_CXXFLAGS) $(BROTLIENC_CFLAGS)