<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/980362337dbf549253c754e4c47ab190ed5c045b">https://github.com/macports/macports-base/commit/980362337dbf549253c754e4c47ab190ed5c045b</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 980362337 Fix portmpkg test failure
</span>980362337 is described below

<span style='display:block; white-space:pre;color:#808000;'>commit 980362337dbf549253c754e4c47ab190ed5c045b
</span>Author: Joshua Root <jmr@macports.org>
AuthorDate: Tue Apr 8 14:53:45 2025 +1000

<span style='display:block; white-space:pre;color:#404040;'>    Fix portmpkg test failure
</span><span style='display:block; white-space:pre;color:#404040;'>    
</span><span style='display:block; white-space:pre;color:#404040;'>    The make_dependency_list test never actually tested making a dependency
</span><span style='display:block; white-space:pre;color:#404040;'>    list for a port with any dependencies, so it's effectively a noop now
</span><span style='display:block; white-space:pre;color:#404040;'>    that the top level port isn't included in the returned list (which is
</span><span style='display:block; white-space:pre;color:#404040;'>    what the actual mpkg code always wanted.)
</span>---
 src/package1.0/tests/portmpkg.test | 22 ++++++++++++----------
 1 file changed, 12 insertions(+), 10 deletions(-)

<span style='display:block; white-space:pre;color:#808080;'>diff --git a/src/package1.0/tests/portmpkg.test b/src/package1.0/tests/portmpkg.test
</span><span style='display:block; white-space:pre;color:#808080;'>index 6913c2f3b..efcecaabe 100644
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/src/package1.0/tests/portmpkg.test
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/src/package1.0/tests/portmpkg.test
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -74,21 +74,23 @@ test make_dependency_list {
</span>     set destination $pwd/destination
 } -body {
     set res [portmpkg::make_dependency_list $subport $destination]
<span style='display:block; white-space:pre;background:#ffe0e0;'>-    if {[llength $res] != 1} {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+    if {[dict size $res] != 0} {
</span>         return "FAIL: unexpectedly long dependency list: $res"
     }
     # indices: 
<span style='display:block; white-space:pre;background:#ffe0e0;'>-    # 0 name
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+    # 0 mport
</span>     # 1 version
     # 2 revision
<span style='display:block; white-space:pre;background:#ffe0e0;'>-    # 3 mport
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    set res [lindex $res 0]
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    if {[llength $res] != 4 ||
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        [lindex $res 0] != $subport ||
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        [lindex $res 1] != 060102 ||
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        [lindex $res 2] != 1} {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-            return "FAIL: incorrect dependency list entry: $res"
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    }
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+    # FIXME: The mocked port has no dependencies, and make_dependency_list
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+    # does not include the top level port, so the result is empty.
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+    #set res_value [dict get $res $some_depname]
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+    #if {[llength $res_value] != 3 ||
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+    #    [lindex $res 1] != 060102 ||
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+    #    [lindex $res 2] != 1} {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+    #        return "FAIL: incorrect dependency list entry: $res"
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+    #}
</span>     return "Dependency list successful."
 
 } -result "Dependency list successful."
</pre><pre style='margin:0'>

</pre>