[patch] bzr diff -r 1501..
Robey Pointer
robey at lag.net
Tue Jan 10 02:58:18 GMT 2006
On 9 Jan 2006, at 17:36, Martin Pool wrote:
> On Mon, 2006-01-09 at 12:43 -0600, John Arbash Meinel wrote:
>
>>> Here's a tiny patch to fix it by turning any missing final revision
>>> spec into an implicit "-1".
>>>
>>> robey
>>
>> The problem is sometimes you want '-1', and sometimes you want
>> WorkingTree. Which is why -r X.. has been defined by the command.
>
> As John says, you should probably do this in the particular command.
> (It does seem like it would be nicer to do it globally though....)
After thought, I agree with both of you -- this particular meaning
(working tree) should be special-cased for diff.
The fix for that turns out to be trivial.
Attached is a new patch, most of which is moving test_diff out of
test_too_much and adding a blackbox test to make sure "diff -r 1.."
works correctly when the working tree is modified. (The one-line
change in diff.py is all that's necessary to get the functionality
working again.)
robey
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: option-patch.txt
Url: https://lists.ubuntu.com/archives/bazaar/attachments/20060109/848bc631/attachment.txt
More information about the bazaar
mailing list