<pre style='margin:0'>
Joshua Root (jmroot) pushed a commit to branch release-2.4
in repository macports-base.
</pre>
<p><a href="https://github.com/macports/macports-base/commit/aced306c3b0756aee8245f07a95dbf49d3225f75">https://github.com/macports/macports-base/commit/aced306c3b0756aee8245f07a95dbf49d3225f75</a></p>
<pre style="white-space: pre; background: #F8F8F8"><span style='display:block; white-space:pre;color:#808000;'>commit aced306c3b0756aee8245f07a95dbf49d3225f75
</span>Author: Joshua Root <jmr@macports.org>
AuthorDate: Thu Feb 2 20:08:48 2017 +1100
<span style='display:block; white-space:pre;color:#404040;'> Apply global_variations when opening port in action_variants
</span><span style='display:block; white-space:pre;color:#404040;'>
</span><span style='display:block; white-space:pre;color:#404040;'> Leaving these out could cause incorrect default variants to be
</span><span style='display:block; white-space:pre;color:#404040;'> displayed for ports that change them based on which variants are
</span><span style='display:block; white-space:pre;color:#404040;'> actually requested.
</span><span style='display:block; white-space:pre;color:#404040;'>
</span><span style='display:block; white-space:pre;color:#404040;'> (cherry picked from commit 9e63a61a75172feea9b825504723c385d2540212)
</span>---
src/port/port.tcl | 11 ++++++++++-
1 file changed, 10 insertions(+), 1 deletion(-)
<span style='display:block; white-space:pre;color:#808080;'>diff --git a/src/port/port.tcl b/src/port/port.tcl
</span><span style='display:block; white-space:pre;color:#808080;'>index 3320fba..3189509 100644
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/src/port/port.tcl
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/src/port/port.tcl
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -3563,6 +3563,15 @@ proc action_variants { action portlist opts } {
</span> }
if {!([info exists options(ports_variants_index)] && $options(ports_variants_index) eq "yes")} {
<span style='display:block; white-space:pre;background:#e0ffe0;'>+ # Add any global_variations to the variations specified for
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ # the port (default variants may change based on this)
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ array unset merged_variations
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ array set merged_variations [array get variations]
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ foreach {variation value} [array get global_variations] {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ if {![info exists merged_variations($variation)]} {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ set merged_variations($variation) $value
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ }
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ }
</span> if {![info exists options(subport)]} {
if {[info exists portinfo(name)]} {
set options(subport) $portinfo(name)
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -3570,7 +3579,7 @@ proc action_variants { action portlist opts } {
</span> set options(subport) $portname
}
}
<span style='display:block; white-space:pre;background:#ffe0e0;'>- if {[catch {set mport [mportopen $porturl [array get options] [array get variations]]} result]} {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ if {[catch {set mport [mportopen $porturl [array get options] [array get merged_variations]]} result]} {
</span> ui_debug "$::errorInfo"
break_softcontinue "Unable to open port: $result" 1 status
}
</pre><pre style='margin:0'>
</pre>