[ann/rfc] bzr-diff-revid

Benjamin Jansen aogail at w007.org
Fri Jan 16 19:01:51 GMT 2009


On Jan 16, 2009, at 12:02 AM, Colin D Bennett wrote:

<snip>

> However, I would not want to see it replace the current, standard diff
> output format.  I consider it an enormously important feature that
> 'bzr diff' produces output compatible with Unix diff.  Non-bzr users
> would certainly be confused by seeing the 'revid:...' if you send them
> the diff, even it the 'patch' command accepted it.

Yeah, I assumed changing the default output would not be acceptable.

> I think a plugin is the proper location for this feature.

Okay, that's fine.

>> [2]
>> In one specific case, the revision's date was "2008-12-30 13:24:25".
>> To actually see the new file contents for the revision, I had to ask
>> bzr for date:"2008-12-31 08:32:32".
>
> That is weird.  Could it be caused by differing time zones?  I think
> this should be looked into.  Could it be a bzr bug?

As you say, time zones are an issue. Based on my, admittedly limited,  
experience, ``bzr diff'' outputs UTC timestmaps, or at least always  
includes an offset. However, when I pass a date: revspec to bzr, it  
seems to want timestamps in local time and ignores the offset field.  
Maybe I am doing something wrong. In any case, I patched Review Board  
to work with those constraints. The real problem came when date: did  
something nonsensical.

In this case, I don't think it is a zone problem, since the minute &  
second fields are completely different than the "real" timestamp. I  
asked about this on #bzr, and was told that date: revspecs have never  
been perfect in bzr. I didn't look into it any further.

- Ben

-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 3914 bytes
Desc: not available
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20090116/de54d6b1/attachment-0001.bin 


More information about the bazaar mailing list