On Sun, Dec 07, 2014 at 07:59:29PM +0100, Daniele Nicolodi wrote: > On 07/12/14 19:54, Zbigniew Jędrzejewski-Szmek wrote: > > On Sun, Dec 07, 2014 at 01:45:13PM -0500, Dave Reisner wrote: > >> On Sun, Dec 07, 2014 at 07:32:38PM +0100, Zbigniew Jędrzejewski-Szmek > >> wrote: > >>> The new frame api should be released in v. 125. > >>> --- > >>> configure.ac | 2 +- > >>> 1 file changed, 1 insertion(+), 1 deletion(-) > >>> > >>> diff --git a/configure.ac b/configure.ac > >>> index 356a3c3d67..94e6a3b54a 100644 > >>> --- a/configure.ac > >>> +++ b/configure.ac > >>> @@ -565,7 +565,7 @@ AM_CONDITIONAL(HAVE_XZ, [test "$have_xz" = "yes"]) > >>> have_lz4=no > >>> AC_ARG_ENABLE(lz4, AS_HELP_STRING([--enable-lz4], [Enable optional LZ4 > >>> support])) > >>> AS_IF([test "x$enable_lz4" == "xyes"], [ > >>> - AC_CHECK_HEADERS(lz4.h, > >>> + PKG_CHECK_MODULES(LZ4, [ liblz4 >= 125 ], > >>> [AC_DEFINE(HAVE_LZ4, 1, [Define in LZ4 is available]) > >>> have_lz4=yes], > >>> [AC_MSG_ERROR([*** LZ4 support requested but headers not > >>> found])]) > >> > >> This is potentially misleading. The headers might be found, but miss the > >> version requirement. We do a similar dance for kmod as we explicitly > >> depend on kmod>=15. > > I don't understand. This checks the pkgconfig file for the version. Why do > > you > > think this might not be enough? > > The error message says that the headers have not be found, but the check > may fail because the installed lz4 version is not recent enough. Ah, OK. I'll fix that.
Zbyszek _______________________________________________ systemd-devel mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/systemd-devel
