PATCH: get rid of /dev/null in diff output

[Robert Collins]

On Mon, 2006-05-15 at 10:24 +0800, James Henstridge wrote:
> The contents of the attached patch can be found in the following branch:
>     http://people.ubuntu.com/~jamesh/bzr/bzr.diff-output/
> 
> This patch changes the "bzr diff" output to include diffs in the patch
> headers, and use the filename plus the epoch date stamp in diffs for
> added or removed files instead of /dev/null.  This stops diffstat from
> getting confused on added/removed files, and should make the output
> less confusing for Windows users where there isn't a /dev/null.
> 
> The change addresses the following launchpad bug:
>     https://launchpad.net/products/bzr/+bug/43033
> 
> When making a diff against a stored revision (as opposed to a working
> tree), I used the commit date of the revision.  Ideally the commit
> date of the last revision to touch the file would be used instead, but
> I'm not sure how to get that information.  Fixing this problem would
> involve altering RevisionTree.get_file_mtime().

repository.get_revision(RevisionTree.inventory[fileid].revision) is the
last modifying revision of the file.

Cheers,
Rob


-- 
GPG key available at: <http://www.robertcollins.net/keys.txt>.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 191 bytes
Desc: This is a digitally signed message part
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20060515/8e09876a/attachment.pgp