<pre style='margin:0'>
Joshua Root (jmroot) pushed a commit to branch master
in repository macports-base.
</pre>
<p><a href="https://github.com/macports/macports-base/commit/9223803e50323607892550b22ec7ab1f9f0b8150">https://github.com/macports/macports-base/commit/9223803e50323607892550b22ec7ab1f9f0b8150</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 9223803e5 portutil: put internal variables in a namespace
</span>9223803e5 is described below
<span style='display:block; white-space:pre;color:#808000;'>commit 9223803e50323607892550b22ec7ab1f9f0b8150
</span>Author: Joshua Root <jmr@macports.org>
AuthorDate: Wed May 1 20:48:30 2024 +1000
<span style='display:block; white-space:pre;color:#404040;'> portutil: put internal variables in a namespace
</span><span style='display:block; white-space:pre;color:#404040;'>
</span><span style='display:block; white-space:pre;color:#404040;'> Deleted target_uniqid since it doesn't seem to be used.
</span>---
src/port1.0/portlint.tcl | 2 +-
src/port1.0/portutil.tcl | 20 ++++++++++----------
2 files changed, 11 insertions(+), 11 deletions(-)
<span style='display:block; white-space:pre;color:#808080;'>diff --git a/src/port1.0/portlint.tcl b/src/port1.0/portlint.tcl
</span><span style='display:block; white-space:pre;color:#808080;'>index 81c74582a..4eaf6bcdf 100644
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/src/port1.0/portlint.tcl
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/src/port1.0/portlint.tcl
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -524,7 +524,7 @@ proc portlint::lint_main {args} {
</span>
###################################################################
<span style='display:block; white-space:pre;background:#ffe0e0;'>- global porturl all_variants patchfiles \
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ global porturl portutil::all_variants patchfiles \
</span> depends_fetch depends_extract depends_patch \
depends_lib depends_build depends_run \
depends_test distfiles fetch.type lint_portsystem \
<span style='display:block; white-space:pre;color:#808080;'>diff --git a/src/port1.0/portutil.tcl b/src/port1.0/portutil.tcl
</span><span style='display:block; white-space:pre;color:#808080;'>index 49f51595f..69a943906 100644
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/src/port1.0/portutil.tcl
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/src/port1.0/portutil.tcl
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -38,10 +38,10 @@ package require macports_util 1.0
</span> package require msgcat
package require porttrace 1.0
<span style='display:block; white-space:pre;background:#ffe0e0;'>-set targets [list]
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-set target_uniqid 0
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-set all_variants [list]
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+namespace eval portutil {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ variable targets [list]
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ variable all_variants [list]
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+}
</span>
########### External High Level Procedures ###########
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -664,7 +664,7 @@ proc variant {args} {
</span> }
# Finally append the ditem to the dlist.
<span style='display:block; white-space:pre;background:#ffe0e0;'>- global all_variants
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ global portutil::all_variants
</span> lappend all_variants $ditem
}
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -688,7 +688,7 @@ proc variant_set {name} {
</span> # variant_remove_ditem name
# Remove variant name's ditem from the all_variants dlist
proc variant_remove_ditem {name} {
<span style='display:block; white-space:pre;background:#ffe0e0;'>- global all_variants
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ global portutil::all_variants
</span> set item_index 0
foreach variant_item $all_variants {
set item_provides [ditem_key $variant_item provides]
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -1656,7 +1656,7 @@ proc recursive_collect_deps {portname {depsfound {}}} \
</span>
proc eval_targets {target} {
<span style='display:block; white-space:pre;background:#ffe0e0;'>- global targets subport version revision portvariants
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ global portutil::targets subport version revision portvariants
</span> set dlist $targets
# the statefile will likely be autocleaned away after install,
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -2034,7 +2034,7 @@ proc canonicalize_variants {variants {sign "+"}} {
</span> }
proc eval_variants {variations} {
<span style='display:block; white-space:pre;background:#ffe0e0;'>- global PortInfo all_variants subport
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ global PortInfo portutil::all_variants subport
</span> set dlist $all_variants
upvar $variations upvariations
lassign [choose_variants $dlist upvariations] chosen negated
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -2121,7 +2121,7 @@ proc eval_variants {variations} {
</span> }
proc check_variants {target} {
<span style='display:block; white-space:pre;background:#ffe0e0;'>- global targets ports_force ports_dryrun PortInfo
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ global portutil::targets ports_force ports_dryrun PortInfo
</span> set result 0
set variations $PortInfo(active_variants)
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -2197,7 +2197,7 @@ proc universal_setup {args} {
</span>
# constructor for target object
proc target_new {name procedure} {
<span style='display:block; white-space:pre;background:#ffe0e0;'>- global targets
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ global portutil::targets
</span> set ditem [ditem_create]
ditem_key $ditem name $name
</pre><pre style='margin:0'>
</pre>