From: Kevin Ballard <kevin@sb.org>
Date: Tue, 28 Sep 2010 00:01:18 GMT
Subject: Re: [PATCH] merge-recursive: option to specify rename threshold
Message-ID: <385B97D7-03F5-4698-A659-15D5D1FA939B@sb.org>
In-Reply-To: <20100927235355.GG11957@burratino>

On Sep 27, 2010, at 4:53 PM, Jonathan Nieder wrote:

> 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