Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1264634

Re: [GIT PULL] tracing: Updates for 4.4

From Steven Rostedt <rostedt@goodmis.org>
Newsgroups linux.kernel
Subject Re: [GIT PULL] tracing: Updates for 4.4
Date 2015-11-07 00:00 +0100
Message-ID <qrY9c-2ke-3@gated-at.bofh.it> (permalink)
References <qrQ1Y-5z3-7@gated-at.bofh.it> <qrWTM-1C2-7@gated-at.bofh.it> <qrXmO-21f-17@gated-at.bofh.it> <qrXwu-25P-5@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On Fri, 6 Nov 2015 14:15:48 -0800
Linus Torvalds <torvalds@linux-foundation.org> wrote:

> On Fri, Nov 6, 2015 at 2:09 PM, Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> >
> > This was reported against linux-next on Nov 2 by Sergey Senozhatsky who
> > supplied a fix patch that I then used as the merge conflict resolution
> > from Nov 3 onward (which everyone involved knew about) ...
> 
> Ok, so the problem is that even though a maintainer is aware of the
> semantic conflict, the pull request doesn't talk about it.

Damn, that was my fault. I simply forgot to mention it :-(


> 
> A lot of maintainers *do* let me know, which I really appreciate, not
> only because it avoids the surprise, but because it happens that I
> miss these semantic conflicts.
> 
> Sometimes the semantic conflict is something that simply doesn't show
> up on x86-64 build I do.  Or I was on the road and didn't do a full
> allmodconfig build. Or it needs very specific config options etc. Or I
> just screw up. When the pull message warns me about it, that will
> help.
> 
> But maybe it's one of those things that isn't written down, and people
> don't think about as part of doing their pull request, so they forget.
> 

Yep, I was going to add that, but I've been working on internal Red Hat
Bugzilla's all week that my mind was not fully focused on that pull
request.

I'll have to add something to my scripts that pulls in any merge
breakage, such that when I run the scripts to generate the pull
request, it adds them in automatically. Because obviously, the manual
approach isn't working for me.

-- Steve
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread


Thread

Re: [GIT PULL] tracing: Updates for 4.4 Linus Torvalds <torvalds@linux-foundation.org> - 2015-11-06 22:40 +0100
  Re: [GIT PULL] tracing: Updates for 4.4 Stephen Rothwell <sfr@canb.auug.org.au> - 2015-11-06 23:10 +0100
    Re: [GIT PULL] tracing: Updates for 4.4 Linus Torvalds <torvalds@linux-foundation.org> - 2015-11-06 23:20 +0100
      Re: [GIT PULL] tracing: Updates for 4.4 Steven Rostedt <rostedt@goodmis.org> - 2015-11-07 00:00 +0100
        Re: [GIT PULL] tracing: Updates for 4.4 Steven Rostedt <rostedt@goodmis.org> - 2015-11-07 00:10 +0100

csiph-web