[136695] trunk/dports/multimedia/mpv/Portfile

[ionic at macports.org]

Revision: 136695
          https://trac.macports.org/changeset/136695
Author:   ionic at macports.org
Date:     2015-05-24 08:27:55 -0700 (Sun, 24 May 2015)
Log Message:
-----------
mpv: use clang as provided by Xcode/CLT on 10.7 and higher.

Should hopefully fix libarclite issue.
Only 10.6 and below should be getting MacPort's clang-3.4 now.
(Maybe) Fixes: #47822.

Modified Paths:
--------------
    trunk/dports/multimedia/mpv/Portfile

Modified: trunk/dports/multimedia/mpv/Portfile
===================================================================
--- trunk/dports/multimedia/mpv/Portfile	2015-05-24 15:25:38 UTC (rev 136694)
+++ trunk/dports/multimedia/mpv/Portfile	2015-05-24 15:27:55 UTC (rev 136695)
@@ -7,7 +7,7 @@
 
 # Please revbump mpv whenever ffmpeg{,-devel} is updated!
 github.setup            mpv-player mpv 0.8.3 v
-revision                2
+revision                3
 categories              multimedia
 license                 GPL-2+
 maintainers             ionic
@@ -108,20 +108,37 @@
 # directory.)
 configure.cppflags-replace  -I${prefix}/include -isystem${prefix}/include
 
-# clang is required to build mpv. The newer, the merrier.
-# Part of getting mpv to work on SL -- c.f. #44662
-compiler.blacklist-append   *gcc* cc
+build.args-append           -v
 
-compiler.fallback-delete    macports-clang-3.4
-compiler.fallback-append    macports-clang-3.4
+platform macosx {
+    if {${os.major} > 10} {
+        # Force recent enough Xcode.
+        # We have to compile with the compiler provided by Xcode/CLT
+        # due to only it shipping libarclite on 10.7 and knowing where
+        # it is.
+        # Could "emulate" the same behavior by using MacPort's clang
+        # and appending -L/usr/lib/arc/ to CFLAGS/CXXFLAGS on 10.7,
+        # but I don't particularily like that idea (yet.)
+        # Keep using the "system compiler" for now. Also means one less
+        # dependency on recent systems, so yay.
+        PortGroup                   xcodeversion 1.0
+        minimum_xcodeversions       {11 4.4}
 
-# Reorder list, move older or unknown versions to the end.
-compiler.fallback-delete    clang macports-clang-3.3
-compiler.fallback-append    clang macports-clang-3.3
+        # Force clang compiler as installed by Xcode's CLT's... hopefully.
+        compiler.blacklist-append   *gcc* cc macports-*
+    } else {
+        # clang is required to build mpv. The newer, the merrier.
+        # Part of getting mpv to work on SL -- c.f. #44662
+        compiler.blacklist-append   *gcc* cc
 
-build.args-append           -v
+        compiler.fallback-delete    macports-clang-3.4
+        compiler.fallback-append    macports-clang-3.4
 
-platform macosx {
+        # Reorder list, move older or unknown versions to the end.
+        compiler.fallback-delete    clang macports-clang-3.3
+        compiler.fallback-append    clang macports-clang-3.3
+    }
+
     configure.args-replace  --disable-apple-remote \
                             --enable-apple-remote
 }
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.macosforge.org/pipermail/macports-changes/attachments/20150524/a1829d92/attachment.html>