<pre style='margin:0'>
Marcus Calhoun-Lopez (MarcusCalhoun-Lopez) pushed a commit to branch master
in repository macports-ports.
</pre>
<p><a href="https://github.com/macports/macports-ports/commit/d27e8d2989e8fcf413e203fa52e7cfd7a3431545">https://github.com/macports/macports-ports/commit/d27e8d2989e8fcf413e203fa52e7cfd7a3431545</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 d27e8d2 xcode_workaround PG: minor refactor
</span>d27e8d2 is described below
<span style='display:block; white-space:pre;color:#808000;'>commit d27e8d2989e8fcf413e203fa52e7cfd7a3431545
</span>Author: Marcus Calhoun-Lopez <mcalhoun@macports.org>
AuthorDate: Mon May 11 07:26:05 2020 -0700
<span style='display:block; white-space:pre;color:#404040;'> xcode_workaround PG: minor refactor
</span>---
_resources/port1.0/group/xcode_workaround-1.0.tcl | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
<span style='display:block; white-space:pre;color:#808080;'>diff --git a/_resources/port1.0/group/xcode_workaround-1.0.tcl b/_resources/port1.0/group/xcode_workaround-1.0.tcl
</span><span style='display:block; white-space:pre;color:#808080;'>index b10f386..c7950ea 100644
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/_resources/port1.0/group/xcode_workaround-1.0.tcl
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/_resources/port1.0/group/xcode_workaround-1.0.tcl
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -47,9 +47,9 @@ proc xcode_workaround::xcode_workaround.appy_fix {} {
</span> use_xcode
# Xcode 11 fixes (applicable to macOS 10.14 and macOS 10.15)
<span style='display:block; white-space:pre;background:#ffe0e0;'>- set apply_fix [ expr ( ${os.major} == 19 || ( ${os.major} == 18 && [vercmp $xcodeversion 11] >= 0 ) ) ]
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ set attempt_fix [ expr ( ${os.major} == 19 || ( ${os.major} == 18 && [vercmp $xcodeversion 11] >= 0 ) ) ]
</span>
<span style='display:block; white-space:pre;background:#ffe0e0;'>- if { ${apply_fix} } {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ if { ${attempt_fix} } {
</span>
# Check if Xcode is newer than defined fixed version
# N.B. vercmp should properly handle none or "" for $xcodeversion or $cltversion
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -57,7 +57,7 @@ proc xcode_workaround::xcode_workaround.appy_fix {} {
</span>
# Check flag from cltversion PG to see if Xcode or CLT should be used
if {${use_xcode}} {
<span style='display:block; white-space:pre;background:#ffe0e0;'>- set apply_fix [expr !${xcode_is_ok}]
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ set attempt_fix [expr !${xcode_is_ok}]
</span> } else {
# Check if CLT version is fixed or not
set clt_is_ok [expr [vercmp $cltversion ${xcode_workaround.fixed_xcode_version}] >= 0]
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -66,16 +66,16 @@ proc xcode_workaround::xcode_workaround.appy_fix {} {
</span> # MacPorts defaults to CLTs, but Xcode can easily be ahead
ui_debug "xcode_workaround: using Xcode since the bug is fixed there"
use_xcode yes
<span style='display:block; white-space:pre;background:#ffe0e0;'>- set apply_fix no
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ set attempt_fix no
</span> } else {
<span style='display:block; white-space:pre;background:#ffe0e0;'>- set apply_fix [expr !${clt_is_ok}]
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ set attempt_fix [expr !${clt_is_ok}]
</span> }
}
}
# Apply the configured fix type
<span style='display:block; white-space:pre;background:#ffe0e0;'>- if { ${apply_fix} } {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ if { ${attempt_fix} } {
</span> switch -- ${xcode_workaround.type} {
append_to_compiler_flags {
# -fno-stack-check workaround only needed on Darwin 19
</pre><pre style='margin:0'>
</pre>