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

[Lukáš Lalinský]

On So, 2007-08-25 at 12:40 -0400, Aaron Bentley wrote:
> Lukáš Lalinský wrote:
> > +def _parse_change_str(revstr):
> > +    if '..' in revstr:
> > +        raise errors.RangeInChangeOption()
> > +    return (revisionspec.RevisionSpec.from_string('before:' + revstr),
> > +            revisionspec.RevisionSpec.from_string(revstr))
> 
> Unfortunately, '..' doesn't always indicate a range.  For example, "bzr
> diff -r branch:../bzr.dev".
> 
> I think you probably need to parse -c as though it was -r and raise an
> error if there are two revision specs in the result.

Thanks, I didn't really realize why is there the regex in
_parse_revision_str.

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.

Lukas

-------------- next part --------------
A non-text attachment was scrubbed...
Name: change.diff
Type: text/x-patch
Size: 15209 bytes
Desc: not available
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20070825/b9427f32/attachment-0001.bin 
-------------- 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/20070825/b9427f32/attachment-0001.pgp