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

Lukáš Lalinský lalinsky at gmail.com
Fri Aug 31 07:56:27 BST 2007


On Pi, 2007-08-31 at 16:48 +1000, Ian Clatworthy wrote:
> Lukáš Lalinský wrote:
> > @@ -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?

Well, I've touched only two commands in this patch, diff and status, and
I wanted to make them react consistently on the same input. I'm not sure
if there are commands (or plugins) that are able to handle revision
ranges with more than 2 revisions. If not, this code could be moved to
option._parse_revision_str.

Lukas

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Toto je =?ISO-8859-1?Q?digit=E1lne?=
	=?ISO-8859-1?Q?_podp=EDsan=E1?= =?UTF-8?Q?_=C4=8Das=C5=A5?=
	=?ISO-8859-1?Q?_spr=E1vy?=
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20070831/6ca17ef4/attachment.pgp 


More information about the bazaar mailing list