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

</pre>
<p><a href="https://github.com/macports/mpbb/commit/1b9a6683a6f23e5cc5e78b6bbec61c2d9585797b">https://github.com/macports/mpbb/commit/1b9a6683a6f23e5cc5e78b6bbec61c2d9585797b</a></p>
<pre style="white-space: pre; background: #F8F8F8"><span style='display:block; white-space:pre;color:#808000;'>commit 1b9a6683a6f23e5cc5e78b6bbec61c2d9585797b
</span>Author: Joshua Root <jmr@macports.org>
AuthorDate: Sun Dec 4 13:18:15 2016 +1100

<span style='display:block; white-space:pre;color:#404040;'>    Don't bother with mportclose or mportshutdown
</span><span style='display:block; white-space:pre;color:#404040;'>    
</span><span style='display:block; white-space:pre;color:#404040;'>    The tool scripts do only read-only operations with the macports API,
</span><span style='display:block; white-space:pre;color:#404040;'>    so nothing needs to be saved, and there's no point deallocating
</span><span style='display:block; white-space:pre;color:#404040;'>    anything as we're just going to exit immediately anyway.
</span>---
 tools/archive-path.tcl       |  1 -
 tools/canonical-variants.tcl | 16 ----------------
 tools/dependencies.tcl       | 19 -------------------
 tools/portgroups.tcl         | 18 ------------------
 tools/supported-archs.tcl    |  1 -
 5 files changed, 55 deletions(-)

<span style='display:block; white-space:pre;color:#808080;'>diff --git a/tools/archive-path.tcl b/tools/archive-path.tcl
</span><span style='display:block; white-space:pre;color:#808080;'>index 4b8039d..1a21e55 100755
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/tools/archive-path.tcl
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/tools/archive-path.tcl
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -69,5 +69,4 @@ if {[info exists portinfo(porturl)]} {
</span>         set workername [ditem_key $mport workername]
         puts [$workername eval get_portimage_path]
     }
<span style='display:block; white-space:pre;background:#ffe0e0;'>-    catch {mportclose $mport}
</span> }
<span style='display:block; white-space:pre;color:#808080;'>diff --git a/tools/canonical-variants.tcl b/tools/canonical-variants.tcl
</span><span style='display:block; white-space:pre;color:#808080;'>index ec2919d..e74461a 100755
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/tools/canonical-variants.tcl
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/tools/canonical-variants.tcl
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -80,19 +80,3 @@ try {
</span> 
 array set info [mportinfo $mport]
 puts $info(canonical_active_variants)
<span style='display:block; white-space:pre;background:#ffe0e0;'>-
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-#try -pass_signal {...}
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-try {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    mportclose $mport
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-} catch {{*} eCode eMessage} {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    ui_warn "mportclose $portname failed: $eMessage"
</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;'>-# shut down MacPorts
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-#try -pass_signal {...}
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-try {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    mportshutdown
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-} catch {{*} eCode eMessage} {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    ui_error "mportshutdown failed: $eMessage"
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    exit 1
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-}
</span><span style='display:block; white-space:pre;color:#808080;'>diff --git a/tools/dependencies.tcl b/tools/dependencies.tcl
</span><span style='display:block; white-space:pre;color:#808080;'>index cf36e5d..e900c74 100755
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/tools/dependencies.tcl
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/tools/dependencies.tcl
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -128,22 +128,3 @@ proc printdependency {ditem} {
</span>     puts [string trim "$depinfo(name) $variantstring"]
 }
 dlist_eval $dlist {} [list printdependency]
<span style='display:block; white-space:pre;background:#ffe0e0;'>-
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-# close all open ports
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-foreach ditem $macports::open_mports {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    #try -pass_signal {...}
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    try {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        mportclose $ditem
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    } catch {{*} eCode eMessage} {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        ui_warn "mportclose [ditem_key $ditem provides] failed: $eMessage"
</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;'>-
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-# shut down MacPorts
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-#try -pass_signal {...}
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-try {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    mportshutdown
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-} catch {{*} eCode eMessage} {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    ui_error "mportshutdown failed: $eMessage"
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    exit 1
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-}
</span><span style='display:block; white-space:pre;color:#808080;'>diff --git a/tools/portgroups.tcl b/tools/portgroups.tcl
</span><span style='display:block; white-space:pre;color:#808080;'>index 81fef77..0d28165 100755
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/tools/portgroups.tcl
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/tools/portgroups.tcl
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -79,21 +79,3 @@ if {[info exists portinfo(portgroups)]} {
</span>         puts "$group-$version"
     }
 }
<span style='display:block; white-space:pre;background:#ffe0e0;'>-
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-# close the Port
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-#try -pass_signal {...}
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-try {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    mportclose $mport
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-} catch {{*} eCode eMessage} {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    ui_error "mportclose $portname failed: $eMessage"
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    exit 1
</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;'>-# shut down MacPorts
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-#try -pass_signal {...}
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-try {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    mportshutdown
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-} catch {{*} eCode eMessage} {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    ui_error "mportshutdown failed: $eMessage"
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    exit 1
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-}
</span><span style='display:block; white-space:pre;color:#808080;'>diff --git a/tools/supported-archs.tcl b/tools/supported-archs.tcl
</span><span style='display:block; white-space:pre;color:#808080;'>index 3ccddf9..00bc786 100755
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/tools/supported-archs.tcl
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/tools/supported-archs.tcl
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -77,5 +77,4 @@ if {[info exists portinfo(porturl)]} {
</span>     } else {
         puts [_mportkey $mport supported_archs]
     }
<span style='display:block; white-space:pre;background:#ffe0e0;'>-    catch {mportclose $mport}
</span> }
</pre><pre style='margin:0'>

</pre>