[80164] trunk/dports/www/firefox-x11-devel/Portfile

[jeremyhu at macports.org]

Revision: 80164
          http://trac.macports.org/changeset/80164
Author:   jeremyhu at macports.org
Date:     2011-07-05 10:43:13 -0700 (Tue, 05 Jul 2011)
Log Message:
-----------
firefox-x11-devel: Adjust compiler choices

Modified Paths:
--------------
    trunk/dports/www/firefox-x11-devel/Portfile

Modified: trunk/dports/www/firefox-x11-devel/Portfile
===================================================================
--- trunk/dports/www/firefox-x11-devel/Portfile	2011-07-05 17:32:24 UTC (rev 80163)
+++ trunk/dports/www/firefox-x11-devel/Portfile	2011-07-05 17:43:13 UTC (rev 80164)
@@ -113,14 +113,18 @@
 
 # xpctools and necko-wifi broke in 3.6b1
 
+# BUILD FIX TODO: test clang in XCode 4.0.x
 # gcc-4.0 has issues, so force gcc-4.2 on Leopard
-# BUILD FIX TODO:
-# llvm-gcc-4.2 and clang fail to build due to bad inline asm as of 2011.06.24
-# This is an error in the package.
+# llvm-gcc-4.2 has trouble with some of the mmx asm
+# recent versions of clang build fine, but older versions were not tested, so
+# not falling forward to clang yet
 if {${configure.compiler} == "gcc-4.0" ||
-    ${configure.compiler} == "llvm-gcc-4.2" ||
-    ${configure.compiler} == "clang"} {
-    configure.compiler gcc-4.2 
+    ${configure.compiler} == "llvm-gcc-4.2"} {
+    configure.compiler gcc-4.2
+    if {![file executable ${configure.cc}]} {
+        depends_build-append port:apple-gcc42
+        configure.compiler apple-gcc-4.2
+    }
 }
 
 # Yes, this is ugly... but the real fix is in the works
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.macosforge.org/pipermail/macports-changes/attachments/20110705/b77a9b5d/attachment.html>