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

Re: Feature Request: Consistent diff

From: Garrett Rooney <rooneg_at_electricjellyfish.net>
Date: 2006-06-16 21:38:15 CEST

On 6/16/06, David Wheeler <david@kineticode.com> wrote:
> On Jun 16, 2006, at 12:19, Garrett Rooney wrote:
>
> > Why bother? If you can run svnlook diff you can also run svn diff
> > with a file:// url to get the same effect. If you want the options
> > that only exist in svn, use svn to get them. I mean I'm not arguing
> > that consistency is bad, just that I don't see it as a real priority
> > in this case since it's perfectly possible to get the same behavior
> > right now with no extra work ;-)
>
> It's mostly for consistency. Also, I'm adding a --diff-options switch
> to SVN::Notify, which only uses svnlook, and I don't want to add code
> to determine the location of svn (even though it's likely to be the
> same). It does enough crazy shit as it is. :-)

Honestly, if you have a way to find svnlook, it strikes me that you've
got something that can be abstracted to a function that lets you find
svn ;-)

> But while I'm on about this, there is one GNU diff option I'd *love*
> to have in Subversion's diff support more than any other: --minimal.
> Or even --ignore-all-space would be useful. It'd make quite a few of
> my diffs a lot smaller.

Ignore space is actually there, or at least there are a few space
related options in svn's diff in 1.4.x. I don't know about minimal
(svn's internal diff is already reasonably minimal), but there's a bug
about implementing non-minimal diffs, since calculating diffs the
current way is often very slow.

-garrett

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Jun 16 21:38:51 2006

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