[MERGE][#56299] New option -C/--change for diff and status to show changes in one revision.

Ian Clatworthy ian.clatworthy at internode.on.net
Fri Aug 31 07:48:56 BST 2007


Lukáš Lalinský wrote:

> Another patch will follow that makes -r branch:..\bzr.dev work, because
> I wouldn't expect to have to use / in local paths on Windows.

bb: approve

> @@ -180,6 +180,10 @@
>              versioned=False):
>          from bzrlib.status import show_tree_status
>  
> +        if revision and len(revision) > 2:
> +            raise errors.BzrCommandError('bzr status --revision takes exactly'
> +                                         ' one or two revision specifiers')
> +
>          tree, file_list = tree_files(file_list)
>              
>          show_tree_status(tree, show_ids=show_ids,

A question: You've changed the status command to throw an exception if
more than 2 revisionspecs are passed. Why only that one? It seems we
could do that more centrally perhaps?

If so, could you move that code as part of the follow-up patch?

Otherwise, all good.

Ian C.



More information about the bazaar mailing list