FYI, this bug (and fix) are still relevant to version 2.2.2

--Jeremy

On Nov 10, 2013, at 17:49, Jeremy Huddleston Sequoia <[email protected]> 
wrote:

> The logic to verify >= clang-3.2 is incorrect:
>  1) It does not correctly check Apple clang versions
>  2) It forces minor version >= 2 even when major version > 3 (eg 4.0 will 
> fail)
> 
> Patch included.
> 
> <clang-check.patch>
> 
> --- Source/autotools/CheckSystemAndBasicDependencies.m4.orig  2013-11-10 
> 17:44:57.000000000 -0800
> +++ Source/autotools/CheckSystemAndBasicDependencies.m4       2013-11-10 
> 17:45:57.000000000 -0800
> @@ -87,7 +87,7 @@ AC_COMPILE_IFELSE([AC_LANG_PROGRAM([
> #endif
> ])], [c_compiler="gcc"], [])
> AC_COMPILE_IFELSE([AC_LANG_PROGRAM([
> -#if !(defined(__clang__) && __clang_major__ >= 3 && __clang_minor__ >= 2)
> +#if !(defined(__clang__) && (__apple_build_version__ >= 4250024 || 
> (!defined(__apple_build_version__) && (__clang_major__ > 3 || 
> (__clang_major__ == 3 && __clang_minor__ >= 2)))))
> #error Not a supported Clang compiler
> #endif
> ])], [c_compiler="clang"], [])
> @@ -106,7 +106,7 @@ AC_COMPILE_IFELSE([AC_LANG_PROGRAM([
> #endif
> ])], [cxx_compiler="g++"], [])
> AC_COMPILE_IFELSE([AC_LANG_PROGRAM([
> -#if !(defined(__clang__) && __clang_major__ >= 3 && __clang_minor__ >= 2)
> +#if !(defined(__clang__) && (__apple_build_version__ >= 4250024 || 
> (!defined(__apple_build_version__) && (__clang_major__ > 3 || 
> (__clang_major__ == 3 && __clang_minor__ >= 2)))))
> #error Not a supported Clang++ compiler
> #endif
> ])], [cxx_compiler="clang++"], [])
> 
> _______________________________________________
> webkit-gtk mailing list
> [email protected]
> https://lists.webkit.org/mailman/listinfo/webkit-gtk

Attachment: smime.p7s
Description: S/MIME cryptographic signature

_______________________________________________
webkit-gtk mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-gtk

Reply via email to