[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