Hi, On 20-02-18 11:00, Gert Doering wrote: > On Tue, Feb 20, 2018 at 10:41:08AM +0100, Gert Doering wrote: >> Your patch has been applied to the master and release/2.4 branch. >> >> Tested on FreeBSD 10, Linux and, indeed, AIX 7 :-) - Linux/gcc is nicely >> clean, FreeBSD/clang finds a few that are silly and want fixing... > > Yeah. Right. And it blows up on centos-6 and opensolaris-10. > > Now I can't claim that I understand this... David double-checked this > macro and m4/ stuff, so "everything should have been good", but it > isn't. > > CentOS 6: > > checking lz4.h usability... no > checking lz4.h presence... no > checking for lz4.h... no > usable LZ4 library or header not found, using version in > src/com > +pat/compat-lz4.* > checking git checkout... yes > ./configure: line 24067: syntax error near unexpected token `newline' > ./configure: line 24067: `AX_CHECK_COMPILE_FLAG(' > make: *** [config.status] Error 2
They both already fail in the step before it seems: > configure.ac:1295: error: Autoconf version 2.64 or higher is required ContOS 6 ships with autoconf 2.63, I don't know what opensolaris-10 has. We could revert to an older version of the macro we ship, or to a simpler (though less clear) AC_CHECK_COMPILE-based check. I'm tempted to ditch the macro and go for the AC_CHECK_COMPILE. -Steffan ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link/slashdot _______________________________________________ Openvpn-devel mailing list Openvpn-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/openvpn-devel