[trafficserver] branch 7.1.x updated: Move Brotli library check into the library check section CFLAGS is getting set the default -g -O2. When optimization is set on CFLAGS or CXXFLAGS, optimization will not be added to the flags.

[zw...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

zwoop pushed a commit to branch 7.1.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/7.1.x by this push:
     new b4cde6a  Move Brotli library check into the library check section CFLAGS is getting set the default -g -O2.  When optimization is set on CFLAGS or CXXFLAGS, optimization will not be added to the flags.
b4cde6a is described below

commit b4cde6a2dcab43ad5067d2e0eca057d1f739b7ac
Author: Bryan Call <bc...@apache.org>
AuthorDate: Tue Nov 7 13:45:04 2017 -0800

    Move Brotli library check into the library check section
    CFLAGS is getting set the default -g -O2.  When optimization is set on CFLAGS
    or CXXFLAGS, optimization will not be added to the flags.
    
    (cherry picked from commit fc69c42e424f4be68467c232dcb0baced6b7d0fc)
---
 configure.ac | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/configure.ac b/configure.ac
index 0dbdd1c..4495d0c 100644
--- a/configure.ac
+++ b/configure.ac
@@ -444,12 +444,6 @@ AC_ARG_WITH([max-threads-per-type],
 )
 AC_SUBST(max_threads_per_type)
 
-# Check Brotli
-AC_CHECK_HEADERS([brotli/encode.h], [has_brotli=1],[has_brotli=0])
-AC_CHECK_LIB([brotlienc],[BrotliEncoderCreateInstance],[AC_SUBST([LIB_BROTLIENC],["-lbrotlienc"])],[has_brotli=0])
-AC_SUBST(has_brotli)
-AM_CONDITIONAL([HAS_BROTLI], [ test "x${has_brotli}" = "x1" ])
-
 #
 # Experimental plugins
 #
@@ -1226,8 +1220,14 @@ if test "x${enable_pcre}" != "xyes"; then
   AC_MSG_ERROR([Cannot find pcre library. Configure --with-pcre=DIR])
 fi
 
-has_backtrace=0
+# Check Brotli
+AC_CHECK_HEADERS([brotli/encode.h], [has_brotli=1],[has_brotli=0])
+AC_CHECK_LIB([brotlienc],[BrotliEncoderCreateInstance],[AC_SUBST([LIB_BROTLIENC],["-lbrotlienc"])],[has_brotli=0])
+AC_SUBST(has_brotli)
+AM_CONDITIONAL([HAS_BROTLI], [ test "x${has_brotli}" = "x1" ])
+
 # Check for backtrace() support
+has_backtrace=0
 AC_CHECK_HEADERS([execinfo.h], [has_backtrace=1],[])
 if test "${has_backtrace}" = "1"; then
   # FreeBSD requires '/usr/ports/devel/libexecinfo' for gdb style backtrace() support

-- 
To stop receiving notification emails like this one, please contact
['"commits@trafficserver.apache.org" <co...@trafficserver.apache.org>'].