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

Re: [PATCH] issue #3071: allow built-in diff to be specified as diff-cmd

From: Augie Fackler <durin42_at_gmail.com>
Date: Wed, 26 Mar 2008 08:32:30 -0500

On Mar 25, 2008, at 3:37 PM, Daniel Shahaf wrote:

> Daniel Danger Bentley wrote on Tue, 25 Mar 2008 at 03:09 -0400:
>> Hmm, I understand the concern about using the empty string to mean
>> "internal
>> diff".
>>
> ...
>> It seems to me what we are discussing is one option (which diff
>> command
>> should be used), and so it makes sense to specify that option in
>> one place.
>>
>
> Makes sense to me too.
>
>> Using "" to mean "built-in" may be confusing. I get that. Another
>> option
>> would be to be "builtin" to mean "built-in", but that 1) prevents
>> running
>> binaries named "builtin"
>
> --diff-cmd=/path/to/builtin
>
> Is there actually a 'builtin' binary anywhere that does diffs?

Just a thought, we could use 'svn' to mean use the built-in diff
command. That sorta makes sense to me, although I'm undecided if the
special case makes sense...

Augie

  • application/pkcs7-signature attachment: smime.p7s
Received on 2008-03-26 14:32:46 CET

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