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

Re: sander's diff library

From: Eric Gillespie <epg_at_pretzelnet.org>
Date: 2003-02-08 20:13:25 CET

Greg Hudson <ghudson@MIT.EDU> writes:

> iteration, but it would be nice.) Is the right answer a --diff-program
> option? A mime-type to diff program mapping in ~/.subversion/config?

Long-term, we want both of those. When we have pluggable diff
programs, users will always have the case where for *just this
run* they want to use a different diff program than what's
specified in the config files. It just so happens that we can
also make use of that code path and command-line option today to
switch between the internal and GNU diff implementations.

So my vote goes to making the internal library the default,
changeable only at run-time with some kind of --diff-program
option.

> Should the existing "-x" option signify an external diff program for
> now? (Should it go away in the long term, to be subsumed by internal
> long options?)

I'm not sure what the answer to this is, but the --diff-program
option will need a way to get command-line arguments to the
program. Whether it should be made of the form --diff-program
'/usr/bin/diff -up' or --diff-program /usr/bin/diff -x -up i am
not sure. I lean toward the former, but i'm sure some people
will cry about spaces in their filenames.

--
Eric Gillespie <*> epg@pretzelnet.org

Build a fire for a man, and he'll be warm for a day. Set a man on
fire, and he'll be warm for the rest of his life. -Terry Pratchett

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Feb 8 20:14:06 2003

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.