<div dir="ltr">Could i rework this patch to add inner ifdef for OSX_VERSION=10.9 to fix build on MacOS X 10.9?<div>All contrib libraries on 10.9 built with clang without any problem.</div></div><div class="gmail_extra"><br>
<br><div class="gmail_quote">On Thu, Nov 7, 2013 at 3:44 PM, Felix Paul Kühne <span dir="ltr"><<a href="mailto:fkuehne@videolan.org" target="_blank">fkuehne@videolan.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Dear Maxim,<br>
<div class="im"><br>
On 07.11.2013, at 11:47, Maxim Bublis <<a href="mailto:b@codemonkey.ru">b@codemonkey.ru</a>> wrote:<br>
<br>
> ---<br>
> contrib/src/main.mak | 4 ++--<br>
> 1 file changed, 2 insertions(+), 2 deletions(-)<br>
><br>
> diff --git a/contrib/src/main.mak b/contrib/src/main.mak<br>
> index 4e568ec..a40cba2 100644<br>
> --- a/contrib/src/main.mak<br>
> +++ b/contrib/src/main.mak<br>
> @@ -97,8 +97,8 @@ endif<br>
><br>
> ifdef HAVE_MACOSX<br>
> MIN_OSX_VERSION=10.6<br>
> -CC=xcrun llvm-gcc-4.2<br>
> -CXX=xcrun llvm-g++-4.2<br>
> +CC=xcrun clang<br>
> +CXX=xcrun clang++<br>
<br>
</div>Technically, this patch is correct. However, we can't merge it as is as there are still some contrib libraries left which simply don't compile with clang, notably GnuTLS and some other weirdos. We need patches to compile those with clang first prior to switching from llvm-gcc to clang.<br>

My "official" 2.1.0 build was created on 10.7 where llvm-gcc-4.2 still exists as part of Xcode 4.6.2 (it was only recently removed in Xcode 5.0).<br>
<br>
Best regards,<br>
<br>
Felix<br>
<div class="HOEnZb"><div class="h5"><br>
_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="https://mailman.videolan.org/listinfo/vlc-devel" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>Maxim Bublis
</div>