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

Re: [PATCH] Experimental svn diff -w

From: Garrett Rooney <rooneg_at_electricjellyfish.net>
Date: 2006-02-03 18:42:10 CET

On 2/3/06, Marc Sherman <msherman@projectile.ca> wrote:
> Peter N. Lundblad wrote:
> >
> > If people like this, I plan to use the -x option for diff/diff3 options,
> > as we do for external diffs and allow GNU diff options, e.g.:
> > svn diff -x '-w'
> >
> > Candidate options are:
> > -i --ignore-case (not sure about this; it has encoding problems)
> > -b --ignore-space-change (for ASCII whitespace)
> > -w --ignore-all-space (dito)
> > --strip-trailing-cr (maybe)
>
> If these options are being handled by the built in diff library,
> shouldn't they be real options to the svn command, rather than parsed
> out of the -x parameter?

No, they should not. We already have precedent for handling diff
options separately from the rest when using external diff tools,
there's no reason to be different for our internal tools. If we did
use regular options, that would mean that our diff options would
pretty much have to be different than those of GNU diff (the most
commonly used external diff tool), which would be pretty annoying for
users.

-garrett

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Feb 3 19:02:50 2006

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.