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

James Henstridge james.henstridge at gmail.com
Mon May 15 03:24:55 BST 2006


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().

James.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: bzr-diff-output.patch
Type: text/x-patch
Size: 13443 bytes
Desc: not available
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20060515/8cebdd38/attachment.bin 


More information about the bazaar mailing list