[MERGE] hide inventory and unknowns commands

John Arbash Meinel john at arbash-meinel.com
Thu Feb 8 14:51:16 GMT 2007


Aaron Bentley wrote:
> Hi all,
> 
> The unknowns command is completely redundant with "bzr ls".  In the
> interest of simplifying the visible command set, I suggest hiding the
> unknowns command, and perhaps removing it later.
> 
> Similarly, the inventory command is redundant with "bzr ls", so I've
> hidden it, too.
> 
> This leaves us with 49 builtin commands by my count.
> 
> I added "See also:" to "unknowns", "modified" and "added".
> 
> Other candidates for hiding are "renames" and "conflicts".  Supplying
> identical functionality would entail adding flags to "status", so I
> think we should do this first.
> 
> On the other hand, perhaps "remove-tree" should become a visible command.
> 
> Aaron

...

     It is also possible to restrict the list of files to a specific
     set. For example: bzr inventory --show-ids this/file
+
+    See also: bzr ls
     """

+    hidden = True
     takes_options = ['revision', 'show-ids', 'kind']
     takes_args = ['file*']

^- I would be fine with actually deprecating 'bzr inventory' in favor of
'bzr ls'.


v- I think we want a couple blank lines here. Also something that we
could completely deprecate.

 class cmd_unknowns(Command):
-    """List unknown files."""
+    """List unknown files.
+
+    See also: "bzr ls --unknown".
+    """
+    hidden = True
     @display_command
     def run(self):
         for f in WorkingTree.open_containing(u'.')[0].unknowns():

John
=:->



More information about the bazaar mailing list