[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-block] [PATCH v6 1/2] bitmaps.md: Convert to rST; move it into

From: Kashyap Chamarthy
Subject: Re: [Qemu-block] [PATCH v6 1/2] bitmaps.md: Convert to rST; move it into 'interop' dir
Date: Tue, 11 Jul 2017 13:49:51 +0200
User-agent: Mutt/ (2016-04-01)

On Tue, Jul 11, 2017 at 06:33:22AM -0500, Eric Blake wrote:
> On 07/11/2017 02:58 AM, Kashyap Chamarthy wrote:


> >> A shame that git rename detection doesn't see these as the same rough
> >> contents, but not too bad. 
> > 
> > _Should_ it detect?
> It depends on the similarities between the file, and on whether you have
> 'git config diff.renames true' set (there are some other parameters you
> can set to fine-tune how much effort git expends on detecting file
> renames, but the defaults are usually good enough).

I see, noted.


> > Indeed -- the things you point out further below were already
> > pre-existing.  
> > 
> > Asking out of curiosity: You say it is nicer split because we'll retain
> > the `git-bisect`-ability?  Or just to keep text motion and actual
> > changes separate?  (I think both.)
> bisect-ability is important on code, but less so on docs. If you were
> doing code motion, I'd absolutely insist that cleanups be separate from
> the motion; but here I don't have a strong preference (separating the
> cleanups DOES make it easier to perform a backport of the cleanup
> without having to backport the rename, but backporting docs tends to be
> less of a priority than backporting code functionality).

Yep, fully agree with you, and I guessed as much.  Given this
understanding, I'll just fix your feedback in this series.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]