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

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