Re: linux-next: manual merge of the msm tree with the arm tree

From: Daniel Walker
Date: Mon Oct 18 2010 - 17:29:59 EST


On Mon, 2010-10-18 at 21:58 +0100, Russell King wrote:
> On Mon, Oct 18, 2010 at 01:12:54PM -0700, Daniel Walker wrote:
> > On Mon, 2010-10-18 at 15:29 -0400, Nicolas Pitre wrote:
> >
> > > > Ok, well in that case why not accept this immediately after the merge
> > > > window? A point when everything is quiet, and most of the tree's are
> > > > empty?
> > >
> > > RMK has his own merge window which closes about at the same time as
> > > Linus' one opens. We thought this was happening last week and therefore
> > > this change was supposed to be the last one.
> >
> > It seems like that could potentially make these kinds of problem worse,
> > since your merging things immediately before sending them to Linus. Like
> > right now we only have a fairly short amount of time to correct this
> > conflict.
>
> What I would like to do in an ideal world is stop merging stuff into my
> tree one week before the merge window opens precisely so that people can
> regression test it. If I were to conform to that, I'd be saying "tough,
> it's your problem" right now. Why?
>
> This change has been discussed at length on the linux-arm-kernel mailing
> list, where patches have been posted and reviewed for it several times
> since July:
>
> http://marc.info/?l=linux-arm-kernel&w=2&r=1&s=late+mdesc&q=b
>
> In September it became ready for merging, and this is what I said:
>
> http://marc.info/?l=linux-arm-kernel&m=128332699032679&w=2
>
> So, there should be absolutely no surprise over it, except from people
> who ignore what's going on in the generic ARM world. No, you can't
> expect people to copy every single board maintainer - we have something
> like 300 odd board files with various maintainers. It's one of the
> reasons that we have mailing lists.

That's why we have get_maintainer.pl, it adds in all the CC's
automatically .. I can notice the thread on the list, but I have no idea
it's modifying my tree unless I examine every patch's diffstat .. To me
it's way more reasonable to ask the patch author to get the CC's right.
That way we all get active notification when something is modifying our
tree's ..

> What I should be saying at this point is "tough, you've got a problem"
> but I won't. I've rewound my tree to drop these changes, because it
> seems several people need more time. You have it - until Tuesday - when
> Nicolas will re-do his patch set on top of whatever my tree is at that
> point.
>
> If you haven't sent me a pull request by Tuesday[*] evening my time, it
> becomes _your_ problem to fix the resulting breakage. A warning: make
> sure your tree is not based on my devel branch - it's regressed to
> 'unstable' state because of this.
>

Ok ..

Daniel

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