<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/ed1bda289afb5caab12e21840c72779aef21bab5">https://github.com/macports/macports-base/commit/ed1bda289afb5caab12e21840c72779aef21bab5</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 ed1bda289 Adjust preferred_hosts implementation
</span>ed1bda289 is described below

<span style='display:block; white-space:pre;color:#808000;'>commit ed1bda289afb5caab12e21840c72779aef21bab5
</span>Author: Joshua Root <jmr@macports.org>
AuthorDate: Wed Oct 23 20:50:34 2024 +1100

<span style='display:block; white-space:pre;color:#404040;'>    Adjust preferred_hosts implementation
</span><span style='display:block; white-space:pre;color:#404040;'>    
</span><span style='display:block; white-space:pre;color:#404040;'>    Use 0 as the reported ping time for hosts in preferred_hosts, since the
</span><span style='display:block; white-space:pre;color:#404040;'>    previously used value of 1 is not guaranteed to be lower than hosts
</span><span style='display:block; white-space:pre;color:#404040;'>    that respond extremely quickly. 0 was previously used to indicate no
</span><span style='display:block; white-space:pre;color:#404040;'>    entry in the cache, so switch to using the empty string for that.
</span><span style='display:block; white-space:pre;color:#404040;'>    
</span><span style='display:block; white-space:pre;color:#404040;'>    Closes: https://github.com/macports/macports-base/pull/351
</span>---
 src/macports1.0/macports.tcl        | 8 ++++----
 src/macports1.0/tests/macports.test | 2 +-
 2 files changed, 5 insertions(+), 5 deletions(-)

<span style='display:block; white-space:pre;color:#808080;'>diff --git a/src/macports1.0/macports.tcl b/src/macports1.0/macports.tcl
</span><span style='display:block; white-space:pre;color:#808080;'>index c3891771e..3906d5040 100644
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/src/macports1.0/macports.tcl
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/src/macports1.0/macports.tcl
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -6296,13 +6296,13 @@ proc macports::get_pingtime {host} {
</span>         variable preferred_hosts
         foreach pattern $preferred_hosts {
             if {[string match -nocase $pattern $host]} {
<span style='display:block; white-space:pre;background:#ffe0e0;'>-                dict set host_cache $host 1
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-                return 1
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+                dict set host_cache $host 0
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+                return 0
</span>             }
         }
<span style='display:block; white-space:pre;background:#ffe0e0;'>-        dict set host_cache $host 0
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+        dict set host_cache $host {}
</span>     }
<span style='display:block; white-space:pre;background:#ffe0e0;'>-    if {[dict get $host_cache $host] != 0} {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+    if {[dict get $host_cache $host] ne {}} {
</span>         return [dict get $host_cache $host]
     }
 
<span style='display:block; white-space:pre;color:#808080;'>diff --git a/src/macports1.0/tests/macports.test b/src/macports1.0/tests/macports.test
</span><span style='display:block; white-space:pre;color:#808080;'>index 4f9b5badd..8c28f0e2d 100644
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/src/macports1.0/tests/macports.test
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/src/macports1.0/tests/macports.test
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -974,7 +974,7 @@ test get_pingtime {
</span>     if {[macports::get_pingtime macports_blacklist] != -1} {
         return "FAIL: wrong time for blacklisted host"
     }
<span style='display:block; white-space:pre;background:#ffe0e0;'>-    if {[macports::get_pingtime macports_pref] != 1} {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+    if {[macports::get_pingtime macports_pref] != 0} {
</span>         return "FAIL: wrong time for preferred host"
     }
     return "Get ping time successful."
</pre><pre style='margin:0'>

</pre>