[ann/rfc] bzr-diff-revid

[Alexander Belchenko]

Benjamin Jansen пишет:
> Hello,
> 
> I've recently been working to get Review Board's [1] bzr support up to 
> snuff. One of the underlying problems is that ``bzr diff'' outputs only 
> timestamps in the diff header. Since Review Board wants to get the full 
> contents of the old file when it loads a diff, I was using a date: 
> revspec to retrieve the file at a particular revision. Unfortunately, I 
> soon ran into at least one revision in our currently small (~250 
> revisions) repository where passing a date: revspec did not retrieve the 
> correct file contents. [2]
> 
> I figured the easiest solution would be to make ``bzr diff'' output 
> revids in the diff header, when a revid is available. Thus, 
> bzr-diff-revid was born. [3] It's a simple hack, but it works for me and 
> made our Review Board integration problems go away. ;)
> 
> For example, with the plugin, ``bzr diff'' outputs diff headers like:
> 
> === modified file 'NEWS'
> --- NEWS    revid:pqm at pqm.ubuntu.com-20090115233242-4bxyn4zcj2a0ksfk
> +++ NEWS    2009-01-16 07:20:35 +0000
> 
> If there is no revid available, the timestamp is used.

Why you don't want to use comments in the diff (i.e. line starting with ===)?