<pre style='margin:0'>
Joshua Root (jmroot) pushed a commit to branch master
in repository macports-ports.

</pre>
<p><a href="https://github.com/macports/macports-ports/commit/9e40a5a4b0d4b5221b55a83642762cf613863ff6">https://github.com/macports/macports-ports/commit/9e40a5a4b0d4b5221b55a83642762cf613863ff6</a></p>
<pre style="white-space: pre; background: #F8F8F8">The following commit(s) were added to refs/heads/master by this push:
<span style='display:block; white-space:pre;color:#404040;'>     new 9e40a5a  compiler_blacklist_versions pg: use base support
</span>9e40a5a is described below

<span style='display:block; white-space:pre;color:#808000;'>commit 9e40a5a4b0d4b5221b55a83642762cf613863ff6
</span>Author: Joshua Root <jmr@macports.org>
AuthorDate: Thu Oct 10 01:38:31 2019 +1100

<span style='display:block; white-space:pre;color:#404040;'>    compiler_blacklist_versions pg: use base support
</span><span style='display:block; white-space:pre;color:#404040;'>    
</span><span style='display:block; white-space:pre;color:#404040;'>    Base has its own proc for getting the version of a compiler as of 2.6,
</span><span style='display:block; white-space:pre;color:#404040;'>    and the result is cached across different portfiles. This does remove
</span><span style='display:block; white-space:pre;color:#404040;'>    support for blacklisting particular versions of apple-gcc-4.2, but
</span><span style='display:block; white-space:pre;color:#404040;'>    nothing in the tree uses that, and nor should it really.
</span>---
 .../group/compiler_blacklist_versions-1.0.tcl      | 43 +---------------------
 1 file changed, 2 insertions(+), 41 deletions(-)

<span style='display:block; white-space:pre;color:#808080;'>diff --git a/_resources/port1.0/group/compiler_blacklist_versions-1.0.tcl b/_resources/port1.0/group/compiler_blacklist_versions-1.0.tcl
</span><span style='display:block; white-space:pre;color:#808080;'>index 3f2b2f2..3a2da7f 100644
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/_resources/port1.0/group/compiler_blacklist_versions-1.0.tcl
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/_resources/port1.0/group/compiler_blacklist_versions-1.0.tcl
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -45,7 +45,7 @@ proc compiler_blacklist_versions._set_compiler_blacklist {option action args} {
</span>             }
             set compiler [lindex ${blacklist} 0]
             set comparisons [lrange ${blacklist} 1 end]
<span style='display:block; white-space:pre;background:#ffe0e0;'>-            set compiler_version [compiler_blacklist_versions._get_compiler_version ${compiler}]
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+            set compiler_version [compiler.command_line_tools_version ${compiler}]
</span>             if {[compiler_blacklist_versions._matches_all_versions ${compiler} ${comparisons}]} {
                 if {${compiler_version} eq ""} {
                     ui_debug "compiler ${compiler} blacklisted because it's not installed or it doesn't work"
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -76,48 +76,9 @@ proc compiler_blacklist_versions._matches_all_versions {compiler comparisons} {
</span> }
 
 proc compiler_blacklist_versions._version_matches {compiler comparison_operator test_version} {
<span style='display:block; white-space:pre;background:#ffe0e0;'>-    set actual_version [compiler_blacklist_versions._get_compiler_version ${compiler}]
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+    set actual_version [compiler.command_line_tools_version ${compiler}]
</span>     if {${actual_version} eq ""} {
         return 1
     }
     return [expr [vercmp ${actual_version} ${test_version}] ${comparison_operator} 0]
 }
<span style='display:block; white-space:pre;background:#ffe0e0;'>-
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-proc compiler_blacklist_versions._get_compiler_version {compiler} {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    global compiler_blacklist_versions._compiler_versions os.major xcodeversion
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    if {[info exists compiler_blacklist_versions._compiler_versions(${compiler})]} {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        return [set compiler_blacklist_versions._compiler_versions(${compiler})]
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    }
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    switch ${compiler} {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        clang {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-            set re {clang(?:_.*)?-([0-9.]+)}
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        }
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        llvm-gcc-4.2 {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-            if {${os.major} > 12 || [vercmp $xcodeversion 5.0] >= 0 || [vercmp $xcodeversion 3.1] < 0} {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-                return ""
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-            }
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-            set re {LLVM build ([0-9.]+)}
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        }
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        gcc-4.2 {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-            if {${os.major} > 11 || [vercmp $xcodeversion 4.2] >= 0} {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-                return ""
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-            }
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-            set re {build ([0-9.]+)}
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        }
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        gcc-4.0 -
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        apple-gcc-4.2 {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-            set re {build ([0-9.]+)}
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        }
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        default {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-            return -code error "don't know how to determine build number of compiler \"${compiler}\""
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        }
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    }
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    set cc [portconfigure::configure_get_compiler cc ${compiler}]
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    if {![file exists ${cc}]} return
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    if {[catch {regexp ${re} [exec ${cc} -v 2>@1] -> compiler_version}]} return
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    if {![info exists compiler_version]} {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        return -code error "couldn't determine build number of compiler \"${compiler}\""
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    }
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    set compiler_blacklist_versions._compiler_versions(${compiler}) ${compiler_version}
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    return ${compiler_version}
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-}
</span></pre><pre style='margin:0'>

</pre>