diff mbox

[maintainer-tools] drm-intel: Add note about patches that move code between files

Message ID 1457523985-6261-1-git-send-email-ander.conselvan.de.oliveira@intel.com (mailing list archive)
State New, archived
Headers show

Commit Message

Ander Conselvan de Oliveira March 9, 2016, 11:46 a.m. UTC
Patches that move big chunks of code between files can cause some
complicated conflicts. Add a note to coordinate with maintainers before
merging such patches.

Signed-off-by: Ander Conselvan de Oliveira <ander.conselvan.de.oliveira@intel.com>
---
 drm-intel.rst | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Dave Gordon March 9, 2016, 12:50 p.m. UTC | #1
On 09/03/16 11:46, Ander Conselvan de Oliveira wrote:
> Patches that move big chunks of code between files can cause some
> complicated conflicts. Add a note to coordinate with maintainers before
> merging such patches.
>
> Signed-off-by: Ander Conselvan de Oliveira <ander.conselvan.de.oliveira@intel.com>
> ---
>   drm-intel.rst | 4 ++++
>   1 file changed, 4 insertions(+)
>
> diff --git a/drm-intel.rst b/drm-intel.rst
> index c0c2f20..89df24b 100644
> --- a/drm-intel.rst
> +++ b/drm-intel.rst
> @@ -350,6 +350,10 @@ An inexhaustive list of details to check:
>     http://developercertificate.org/. dim apply-branch should do this
>     automatically for you.
>
> +* For patches that move around lots of code (file rename or extraction) please
> +  coordinate with maintainers to avoid unnecessary pain with conflicts. Usually
> +  some explicit merges are needed to avoid git getting lost.
> +
>   On Confidence, Complexity, and Transparency
>   -------------------------------------------

Perhaps also add a note about "git blame -C", as recently discussed here 
by Tvrtko & Arun in the thread "[PATCH 044/190] drm/i915: Move GEM 
request routines to i915_gem_request.c"?

.Dave.
Ander Conselvan de Oliveira March 10, 2016, 8:34 a.m. UTC | #2
On Wed, 2016-03-09 at 12:50 +0000, Dave Gordon wrote:
> On 09/03/16 11:46, Ander Conselvan de Oliveira wrote:
> > Patches that move big chunks of code between files can cause some
> > complicated conflicts. Add a note to coordinate with maintainers before
> > merging such patches.
> > 
> > Signed-off-by: Ander Conselvan de Oliveira <
> > ander.conselvan.de.oliveira@intel.com>
> > ---
> >   drm-intel.rst | 4 ++++
> >   1 file changed, 4 insertions(+)
> > 
> > diff --git a/drm-intel.rst b/drm-intel.rst
> > index c0c2f20..89df24b 100644
> > --- a/drm-intel.rst
> > +++ b/drm-intel.rst
> > @@ -350,6 +350,10 @@ An inexhaustive list of details to check:
> >     http://developercertificate.org/. dim apply-branch should do this
> >     automatically for you.
> > 
> > +* For patches that move around lots of code (file rename or extraction)
> > please
> > +  coordinate with maintainers to avoid unnecessary pain with conflicts.
> > Usually
> > +  some explicit merges are needed to avoid git getting lost.
> > +
> >   On Confidence, Complexity, and Transparency
> >   -------------------------------------------
> 
> Perhaps also add a note about "git blame -C", as recently discussed here 
> by Tvrtko & Arun in the thread "[PATCH 044/190] drm/i915: Move GEM 
> request routines to i915_gem_request.c"?

I think that would be a bit out of place here. What this patch changes is the
list of things to check before merging a patch. It doesn't discuss the merits of
moving code around and is not a dump for git tips and tricks. To keep the above
list concise and on-topic, I'd rather leave that out.

I didn't know about 'git blame -C', however. Maybe we could have a list of
useful git commands somewhere? Probably out of maintainer tools if it would
include information that is useful when not merging patches. I'm not sure I
would have read that list if it existed though. Last time I remember learning so
mething new about git was when I stumbled upon git commit --fixup in Damien's
blog, but that was a fairly random event.

Ander
diff mbox

Patch

diff --git a/drm-intel.rst b/drm-intel.rst
index c0c2f20..89df24b 100644
--- a/drm-intel.rst
+++ b/drm-intel.rst
@@ -350,6 +350,10 @@  An inexhaustive list of details to check:
   http://developercertificate.org/. dim apply-branch should do this
   automatically for you.
 
+* For patches that move around lots of code (file rename or extraction) please
+  coordinate with maintainers to avoid unnecessary pain with conflicts. Usually
+  some explicit merges are needed to avoid git getting lost.
+
 On Confidence, Complexity, and Transparency
 -------------------------------------------