[PATCH 2/2] erofs-utils: set up all compiler/linker variables independently
Gao Xiang
gaoxiang25 at huawei.com
Tue Nov 12 22:26:50 AEDT 2019
Otherwise, the following checking will be effected
and it can cause unexpected behavior on configuring.
Founded by the upcoming XZ algorithm patches.
Signed-off-by: Gao Xiang <gaoxiang25 at huawei.com>
---
configure.ac | 35 ++++++++++++++++++-----------------
1 file changed, 18 insertions(+), 17 deletions(-)
diff --git a/configure.ac b/configure.ac
index a93767f61578..a5adf172cc43 100644
--- a/configure.ac
+++ b/configure.ac
@@ -131,7 +131,7 @@ if test "x$enable_lz4" = "xyes"; then
test -z "${with_lz4_libdir}" || LZ4_LIBS="-L$with_lz4_libdir $LZ4_LIBS"
saved_CPPFLAGS=${CPPFLAGS}
- CPPFLAGS="${LZ4_CFLAGS} ${CFLAGS}"
+ CPPFLAGS="${LZ4_CFLAGS} ${CPPFLAGS}"
AC_CHECK_HEADERS([lz4.h],[have_lz4h="yes"], [])
@@ -150,28 +150,29 @@ if test "x$enable_lz4" = "xyes"; then
])
], [AC_MSG_ERROR([Cannot find proper lz4 version (>= 1.8.0)])])
LDFLAGS=${saved_LDFLAGS}
-
- if test "x${have_lz4}" = "xyes"; then
- AC_DEFINE([LZ4_ENABLED], [1], [Define to 1 if lz4 is enabled.])
-
- if test "x${have_lz4hc}" = "xyes"; then
- AC_DEFINE([LZ4HC_ENABLED], [1], [Define to 1 if lz4hc is enabled.])
- fi
-
- if test "x${lz4_force_static}" = "xyes"; then
- LDFLAGS="-all-static ${LDFLAGS}"
- else
- test -z "${with_lz4_libdir}" || LZ4_LIBS="-R ${with_lz4_libdir} $LZ4_LIBS"
- fi
- LIBS="$LZ4_LIBS $LIBS"
- fi
fi
- CFLAGS=${saved_CPPFLAGS}
+ CPPFLAGS=${saved_CPPFLAGS}
fi
+# Set up needed symbols, conditionals and compiler/linker flags
AM_CONDITIONAL([ENABLE_LZ4], [test "x${have_lz4}" = "xyes"])
AM_CONDITIONAL([ENABLE_LZ4HC], [test "x${have_lz4hc}" = "xyes"])
+if test "x${have_lz4}" = "xyes"; then
+ AC_DEFINE([LZ4_ENABLED], [1], [Define to 1 if lz4 is enabled.])
+
+ if test "x${have_lz4hc}" = "xyes"; then
+ AC_DEFINE([LZ4HC_ENABLED], [1], [Define to 1 if lz4hc is enabled.])
+ fi
+
+ if test "x${lz4_force_static}" = "xyes"; then
+ LDFLAGS="-all-static ${LDFLAGS}"
+ else
+ test -z "${with_lz4_libdir}" || LZ4_LIBS="-R ${with_lz4_libdir} $LZ4_LIBS"
+ fi
+ LIBS="$LZ4_LIBS $LIBS"
+fi
+
AC_CONFIG_FILES([Makefile
man/Makefile
lib/Makefile
--
2.17.1
More information about the Linux-erofs
mailing list