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
