<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/96d1218166283bb0fc8a77c19a08d902ff7efc48">https://github.com/macports/macports-base/commit/96d1218166283bb0fc8a77c19a08d902ff7efc48</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 96d121816 Move precious_dirs check out of _deactivate_file
</span>96d121816 is described below
<span style='display:block; white-space:pre;color:#808000;'>commit 96d1218166283bb0fc8a77c19a08d902ff7efc48
</span>Author: Joshua Root <jmr@macports.org>
AuthorDate: Thu Jun 20 01:38:34 2024 +1000
<span style='display:block; white-space:pre;color:#404040;'> Move precious_dirs check out of _deactivate_file
</span><span style='display:block; white-space:pre;color:#404040;'>
</span><span style='display:block; white-space:pre;color:#404040;'> Don't add the precious dirs to the list in _deactivate_contents in the
</span><span style='display:block; white-space:pre;color:#404040;'> first place.
</span>---
src/registry2.0/portimage.tcl | 16 ++++++++--------
1 file changed, 8 insertions(+), 8 deletions(-)
<span style='display:block; white-space:pre;color:#808080;'>diff --git a/src/registry2.0/portimage.tcl b/src/registry2.0/portimage.tcl
</span><span style='display:block; white-space:pre;color:#808080;'>index 539f52b9b..5dd7ad3f6 100644
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/src/registry2.0/portimage.tcl
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/src/registry2.0/portimage.tcl
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -813,13 +813,8 @@ proc _deactivate_file {dstfile} {
</span> } elseif { $filetype eq "directory" } {
# 0 item means empty.
if { [llength [readdir $dstfile]] == 0 } {
<span style='display:block; white-space:pre;background:#ffe0e0;'>- variable precious_dirs
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- if {![dict exists $precious_dirs $dstfile]} {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- ui_debug "deactivating directory: $dstfile"
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- ::file delete -- $dstfile
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- } else {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- ui_debug "directory $dstfile does not belong to us"
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- }
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ ui_debug "deactivating directory: $dstfile"
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ ::file delete -- $dstfile
</span> } else {
ui_debug "$dstfile is not empty"
}
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -840,6 +835,7 @@ proc _deactivate_contents {port imagefiles {force 0} {rollback 0}} {
</span> _progress start
set seendirs [dict create]
<span style='display:block; white-space:pre;background:#e0ffe0;'>+ variable precious_dirs
</span> foreach file $imagefiles {
incr progress_step
_progress update $progress_step $progress_total_steps
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -860,8 +856,12 @@ proc _deactivate_contents {port imagefiles {force 0} {rollback 0}} {
</span> # Split out the filename's subpaths and add them to the image list
# as well.
while {![dict exists $seendirs $directory]} {
<span style='display:block; white-space:pre;background:#ffe0e0;'>- lappend files $directory
</span> dict set seendirs $directory 1
<span style='display:block; white-space:pre;background:#e0ffe0;'>+ if {[dict exists $precious_dirs $directory]} {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ ui_debug "directory $directory does not belong to us"
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ break
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ }
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ lappend files $directory
</span> incr progress_total_steps
set directory [::file dirname $directory]
}
</pre><pre style='margin:0'>
</pre>