[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: Too much information in diff header?

From: Julian Foad <julianfoad_at_btopenworld.com>
Date: Mon, 07 Jul 2008 21:06:25 +0100

Karl Fogel wrote:
(re. removing the date field)
> Oh, lots of people use wide consoles. But still, we should try to
> conserve horizontal space -- no matter how wide your console, there can
> always be a path that's too long for it. Minimizing the number of such
> paths is still a worthy goal.

Yes, it is a worthy goal in general. But do you have a particular
problem with this that's arisen recently? This format is (I believe)
mimicking what standard "diff" and probably CVS diff does, and the
information is useful to humans perusing a patch where the revision
numbers might be pretty meaningless to them (not everbody memorises
revision numbers like some of us do), and there may well be applications
that are colouring or otherwise parsing this output that might break.

I would have thought it would be better to leave well alone at this
stage of development, unless and until an occasion arises when we are
overhauling the format of Subversion diff output anyway (e.g. if we
change it as part of "svn patch" work).

Plenty of things are broken in "svn diff", and this isn't one of them.

However, I'll only say -0 if you really want to change it.

Just my tuppence worth.
- Julian

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-07-07 22:06:41 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.