Zbigniew Jędrzejewski-Szmek | 5d6eedd | 2017-01-31 12:11:17 -0500 | [diff] [blame] | 1 | From 25cd9391929a46f1f38daf6aa4421de6a1afc47e Mon Sep 17 00:00:00 2001 |
Zbigniew Jędrzejewski-Szmek | 03e93e2 | 2017-01-29 17:22:41 -0500 | [diff] [blame] | 2 | From: =?UTF-8?q?Zbigniew=20J=C4=99drzejewski-Szmek?= <zbyszek@in.waw.pl> |
| 3 | Date: Wed, 23 Nov 2016 10:18:30 -0500 |
Jan Synacek | eaf2f30 | 2017-01-16 13:01:41 +0100 | [diff] [blame] | 4 | Subject: [PATCH] build-sys: check for lz4 in the old and new numbering scheme |
| 5 | (#4717) |
| 6 | |
| 7 | lz4 upstream decided to switch to an incompatible numbering scheme |
Zbigniew Jędrzejewski-Szmek | 03e93e2 | 2017-01-29 17:22:41 -0500 | [diff] [blame] | 8 | (1.7.3 follows 131, to match the so version). |
| 9 | PKG_CHECK_MODULES does not allow two version matches for the same package, |
| 10 | so e.g. lz4 < 10 || lz4 >= 125 cannot be used. Check twice, once for |
| 11 | "new" numbers (anything below 10 is assume to be new), once for the "old" |
| 12 | numbers (anything above >= 125). This assumes that the "new" versioning |
| 13 | will not get to 10 to quickly. I think that's a safe assumption, lz4 is a |
| 14 | mature project. |
Jan Synacek | eaf2f30 | 2017-01-16 13:01:41 +0100 | [diff] [blame] | 15 | |
| 16 | Fixed #4690. |
Zbigniew Jędrzejewski-Szmek | 03e93e2 | 2017-01-29 17:22:41 -0500 | [diff] [blame] | 17 | (cherry picked from commit 3d4cf7de48a74726694abbaa09f9804b845ff3ba) |
Jan Synacek | eaf2f30 | 2017-01-16 13:01:41 +0100 | [diff] [blame] | 18 | --- |
| 19 | configure.ac | 9 ++++++--- |
| 20 | 1 file changed, 6 insertions(+), 3 deletions(-) |
| 21 | |
| 22 | diff --git a/configure.ac b/configure.ac |
Zbigniew Jędrzejewski-Szmek | 03e93e2 | 2017-01-29 17:22:41 -0500 | [diff] [blame] | 23 | index 0b10fc7de7..1928e65bde 100644 |
Jan Synacek | eaf2f30 | 2017-01-16 13:01:41 +0100 | [diff] [blame] | 24 | --- a/configure.ac |
| 25 | +++ b/configure.ac |
Zbigniew Jędrzejewski-Szmek | 03e93e2 | 2017-01-29 17:22:41 -0500 | [diff] [blame] | 26 | @@ -623,10 +623,13 @@ AM_CONDITIONAL(HAVE_BZIP2, [test "$have_bzip2" = "yes"]) |
Jan Synacek | eaf2f30 | 2017-01-16 13:01:41 +0100 | [diff] [blame] | 27 | have_lz4=no |
| 28 | AC_ARG_ENABLE(lz4, AS_HELP_STRING([--disable-lz4], [Disable optional LZ4 support])) |
| 29 | AS_IF([test "x$enable_lz4" != "xno"], [ |
| 30 | - PKG_CHECK_MODULES(LZ4, [ liblz4 >= 125 ], |
| 31 | - [AC_DEFINE(HAVE_LZ4, 1, [Define in LZ4 is available]) |
| 32 | + PKG_CHECK_MODULES(LZ4, [ liblz4 < 10 ], |
| 33 | + [AC_DEFINE(HAVE_LZ4, 1, [Define if LZ4 is available]) |
| 34 | have_lz4=yes], |
| 35 | - have_lz4=no) |
| 36 | + [PKG_CHECK_MODULES(LZ4, [ liblz4 >= 125 ], |
| 37 | + [AC_DEFINE(HAVE_LZ4, 1, [Define if LZ4 is available]) |
| 38 | + have_lz4=yes], |
| 39 | + have_lz4=no)]) |
| 40 | AS_IF([test "x$have_lz4" = xno -a "x$enable_lz4" = xyes], |
| 41 | [AC_MSG_ERROR([*** LZ4 support requested but libraries not found])]) |
| 42 | ]) |