Re: [PATCH] merge-recursive: option to specify rename threshold
On Sep 27, 2010, at 4:53 PM, Jonathan Nieder wrote:
Show 18 quoted lines
> Kevin Ballard wrote:
>
>> After taking a look at this, it raises another question. -B, -M, and
>> -C all have optional arguments, but the long-form names don't seem
>> to support that.
> [...]
>> if I make it support `git diff
>> --rename-threshold foo` that would also work, but the name doesn't
>> seem appropriate without the argument.
>
> Right --- with merge-recursive the argument doesn't need to be
> optional, but with git diff it does.
>
> How about
>
> --detect-renames=<threshold>
> --detect-copies=<threshold>
> --detect-rewrites=<threshold>/<threshold>
Good timing, I just sent out a patch that does almost exactly this, though I went with --break-rewrites instead of --detect-rewrites.
-Kevin Ballard