Older LZ4 library versions used a version number > 100 and not the
current x.y.z versioning scheme. This results in version 122 being
numberically higher than the check we have liblz4 > 1.7.1. And
since that old version (122) does not have the LZ4_compress_default(),
the building explodes later on.
This patch enhances the version check to also ensure the version
number is lower than 100. In addition the function checking we
had was not triggered if system library was found via pkg-config,
so this have now been reworked to really check if we have at least
two of the most important LZ4 functions - as long as a system
library have been found or been accepted via the LZ4_{CFLAGS,LIBS}
variables.
There are more ways to check for functions in autoconf. I opted
for AC_CHECK_LIB() instead of AC_CHECK_FUNC{,S}() as the latter
ones does not test if a function exists in a specific library. This
have the downside of needing to tests instead of AC_CHECK_FUNCS()
which could test for more functions in one go. We also do not
overwrite the LZ4_LIBS variable on success, as that could change
already set library paths (-L)
Finally, a stupid typo got fixed as well.
Trac: 939
Signed-off-by: David Sommerseth <davids@openvpn.net>
Tested-by: Richard Bonhomme <fragmentux@gmail.com>
Acked-by: Gert Doering <gert@greenie.muc.de>
Message-Id: <20171002161812.9376-1-davids@openvpn.net>
URL: https://www.mail-archive.com/search?l=mid&q=20171002161812.9376-1-davids@openvpn.net
Signed-off-by: Gert Doering <gert@greenie.muc.de>
... | ... |
@@ -1091,7 +1091,7 @@ if test "$enable_lz4" = "yes" && test "$enable_comp_stub" = "no"; then |
1091 | 1091 |
if test -z "${LZ4_CFLAGS}" -a -z "${LZ4_LIBS}"; then |
1092 | 1092 |
# if the user did not explicitly specify flags, try to autodetect |
1093 | 1093 |
PKG_CHECK_MODULES([LZ4], |
1094 |
- [liblz4 >= 1.7.1], |
|
1094 |
+ [liblz4 >= 1.7.1 liblz4 < 100], |
|
1095 | 1095 |
[have_lz4="yes"], |
1096 | 1096 |
[] # If this fails, we will do another test next |
1097 | 1097 |
) |
... | ... |
@@ -1131,16 +1131,20 @@ if test "$enable_lz4" = "yes" && test "$enable_comp_stub" = "no"; then |
1131 | 1131 |
fi |
1132 | 1132 |
fi |
1133 | 1133 |
|
1134 |
- # if LZ4_LIBS is set, we assume it will work, otherwise test |
|
1135 |
- if test -z "${LZ4_LIBS}"; then |
|
1134 |
+ # Double check we have a few needed functions |
|
1135 |
+ if test "${have_lz4}" = "yes" ; then |
|
1136 | 1136 |
AC_CHECK_LIB([lz4], |
1137 |
- [LZ4_compress], |
|
1138 |
- [LZ4_LIBS="-llz4"], |
|
1137 |
+ [LZ4_compress_default], |
|
1138 |
+ [], |
|
1139 |
+ [have_lz4="no"]) |
|
1140 |
+ AC_CHECK_LIB([lz4], |
|
1141 |
+ [LZ4_decompress_safe], |
|
1142 |
+ [], |
|
1139 | 1143 |
[have_lz4="no"]) |
1140 | 1144 |
fi |
1141 | 1145 |
|
1142 | 1146 |
if test "${have_lz4}" != "yes" ; then |
1143 |
- AC_MSG_RESULT([ usuable LZ4 library or header not found, using version in src/compat/compat-lz4.*]) |
|
1147 |
+ AC_MSG_RESULT([ usable LZ4 library or header not found, using version in src/compat/compat-lz4.*]) |
|
1144 | 1148 |
AC_DEFINE([NEED_COMPAT_LZ4], [1], [use copy of LZ4 source in compat/]) |
1145 | 1149 |
LZ4_LIBS="" |
1146 | 1150 |
fi |