<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/ffbd62ad716d7f4c4e96d97f31a844445ca13832">https://github.com/macports/macports-base/commit/ffbd62ad716d7f4c4e96d97f31a844445ca13832</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 ffbd62ad7 restore::deactivate_all: deactivate in order again
</span>ffbd62ad7 is described below

<span style='display:block; white-space:pre;color:#808000;'>commit ffbd62ad716d7f4c4e96d97f31a844445ca13832
</span>Author: Joshua Root <jmr@macports.org>
AuthorDate: Tue Jun 11 14:52:50 2024 +1000

<span style='display:block; white-space:pre;color:#404040;'>    restore::deactivate_all: deactivate in order again
</span><span style='display:block; white-space:pre;color:#404040;'>    
</span><span style='display:block; white-space:pre;color:#404040;'>    Although deactivating everything means we don't have to worry about
</span><span style='display:block; white-space:pre;color:#404040;'>    leaving ports in a broken state due to missing deps, some pre- and
</span><span style='display:block; white-space:pre;color:#404040;'>    post-deactivate code that uses things from the deps can still fail if
</span><span style='display:block; white-space:pre;color:#404040;'>    the deps are deactivated first.
</span>---
 src/macports1.0/restore.tcl | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

<span style='display:block; white-space:pre;color:#808080;'>diff --git a/src/macports1.0/restore.tcl b/src/macports1.0/restore.tcl
</span><span style='display:block; white-space:pre;color:#808080;'>index 46eab21a9..64cb13366 100644
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/src/macports1.0/restore.tcl
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/src/macports1.0/restore.tcl
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -255,12 +255,11 @@ namespace eval restore {
</span>     # warnings as possible will be printed.
     proc deactivate_all {} {
         set options [dict create ports_nodepcheck 1 ports_force 1]
<span style='display:block; white-space:pre;background:#ffe0e0;'>-        foreach port [registry::entry installed] {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-            if {![registry::run_target $port deactivate $options]} {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-                if {[catch {portimage::deactivate [$port name] [$port version] [$port revision] [$port variants] $options} result]} {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-                    ui_debug $::errorInfo
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-                    ui_warn "Failed to deactivate [$port name]: $result"
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-                }
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+        foreach port [deactivation_order [registry::entry installed]] {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+            if {![registry::run_target $port deactivate $options]
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+                && [catch {portimage::deactivate [$port name] [$port version] [$port revision] [$port variants] $options} result]} {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+                ui_debug $::errorInfo
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+                ui_warn "Failed to deactivate [$port name]: $result"
</span>             }
         }
     }
</pre><pre style='margin:0'>

</pre>