[MERGE] add modification dates to diff output, get rid of /dev/null (bug #43033)

[John A Meinel]

Aaron Bentley wrote:
> Robert Collins wrote:
> | On Wed, 2006-05-24 at 14:36 +0800, James Henstridge wrote:
> |
> |>>The code changes themselves look clean, and the test cases look
> |>>sufficient. So if we decide we really do want the timestamps, you
> |>
> |>have
> |>
> |>>my +1.
> |
> |
> | It has my +1 too. Lets say we need a third +1 to get a
> | majority-of-committers on this? (Given the ui considerations).
> |
> | Aaron ? Martin ?
> 
> Technically, it has my +1.  Policy-wise, I'll abstain.
> 
> Aaron

I think Aaron and I are both +1 on the code level change. But haven't
decided if we want timestamps in the output. And I've been wanting to
just wait a little bit to see if any discussion comes up.

I'm pretty middle of the road, so if adding timestamps makes it easier
to apply with patch, I guess I'm overall positive on it.

Note that it will probably slightly effect revision bundles. I'm not
positive, but I know we expect to be able to round trip nano-second
precision timestamps. So we have to include the high-precision
somewhere, and then we will also be displaying a low-precision one for
each file.

John
=:->

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 249 bytes
Desc: OpenPGP digital signature
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20060601/bcc17365/attachment.pgp