[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: Justin Erenkrantz <jerenkrantz_at_apache.org>
Date: 2003-02-06 06:17:28 CET

--On Wednesday, February 05, 2003 19:56:13 -0600 Ben Collins-Sussman
<sussman@collab.net> wrote:

> Justin Erenkrantz <jerenkrantz@apache.org> writes:
>
>> Compile-time option, but certainly not run-time.
>
> Why not? I really want it to be run-time.
>
> Highest bar to experimentation == 'svn switch; make'
> Medium bar to experimentation == './configure --with-internal-diff;
> make'
> Lowest bar to experimentation == 'svn diff --internal-diff'

Adding a switch at run-time is code that we would never want to use
permanently. Adding a temporary flag irks me somewhat.

Furthermore, I believe doing this sort of experimentation would discourage
long-term experimentation with Sander's diff library. If you select the
diff library at configure-time, you don't have to actively remember to pass
the flag. What we're looking for is long-term experimentation from people
who would realize a broken diff when they see it.

Passing an --internal-diff and telling everyone to sometimes use it isn't
going to help. You actually have to use it all the time - we've got the
common cases already down (via make test). Adding a configure flag and
telling those people who aren't afraid to start eating Sander's delicious
dog food would shake out the bugs a lot quicker.

This is tending towards a bikeshed though. I'm just stating that I believe
a run-time flag isn't going to advance the cause. -- justin

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Feb 6 06:18:14 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.