[121485] trunk/base/src/port/port.tcl

cal at macports.org cal at macports.org
Thu Jun 26 14:43:04 PDT 2014


Revision: 121485
          https://trac.macports.org/changeset/121485
Author:   cal at macports.org
Date:     2014-06-26 14:43:03 -0700 (Thu, 26 Jun 2014)
Log Message:
-----------
base: port/port.tcl: Fix a problem caused by a flag that didn't get reset in batch mode, closes #44091

Modified Paths:
--------------
    trunk/base/src/port/port.tcl

Modified: trunk/base/src/port/port.tcl
===================================================================
--- trunk/base/src/port/port.tcl	2014-06-26 21:23:59 UTC (rev 121484)
+++ trunk/base/src/port/port.tcl	2014-06-26 21:43:03 UTC (rev 121485)
@@ -4594,6 +4594,16 @@
         # What kind of arguments does the command expect?
         set expand [action_needs_portlist $action]
 
+        # (Re-)initialize private_options(ports_no_args) to no, because it might still be yes
+        # from the last command in batch mode. If we don't do this, port will fail to
+        # distinguish arguments that expand to empty lists from no arguments at all:
+        # > installed
+        # > list outdated
+        # will then behave like
+        # > list
+        # if outdated expands to the empty list. See #44091, which was filed about this.
+        set private_options(ports_no_args) "no"
+
         # Parse action arguments, setting a special flag if there were none
         # We otherwise can't tell the difference between arguments that evaluate
         # to the empty set, and the empty set itself.
@@ -4601,7 +4611,7 @@
         switch -- [lookahead] {
             ;       -
             _EOF_ {
-                set private_options(ports_no_args) yes
+                set private_options(ports_no_args) "yes"
             }
             default {
                 if {[ACTION_ARGS_NONE] == $expand} {
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.macosforge.org/pipermail/macports-changes/attachments/20140626/1adabf92/attachment.html>


More information about the macports-changes mailing list