From: Len Brown <lenb@kernel.org>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Daniel Lezcano <daniel.lezcano@linaro.org>,
Robert Lee <rob.lee@linaro.org>,
linux-acpi <linux-acpi@vger.kernel.org>,
Linux PM list <linux-pm@vger.kernel.org>,
linux-kernel <linux-kernel@vger.kernel.org>
Subject: Re: [GIT PULL] ACPI & Power Management patches for Linux-3.4-merge
Date: Fri, 30 Mar 2012 15:54:23 -0400 [thread overview]
Message-ID: <4F760F6F.5060002@kernel.org> (raw)
In-Reply-To: <CA+55aFx8XT0biUUWfTRf1VJC_2dgcqV5Lr3AxQG0ze+B0gSC9A@mail.gmail.com>
I'll take curtain C:-)
Will send you a fresh merge request in about an hour.
Sorry for the trouble.
thanks,
-Len
On 03/30/2012 03:32 PM, Linus Torvalds wrote:
> On Fri, Mar 30, 2012 at 12:16 PM, Linus Torvalds
> <torvalds@linux-foundation.org> wrote:
>>
>> But there's been at least three merges that submaintainers did for me
>> this merge window where I looked at their merge and said "No, that's
>> wrong, and I would have done it better". Two of those were the nice
>> kind of "I left it unmerged, but here's my example merge if you want
>> to take it", so the wrong merges didn't ever show up in the tree. But
>> yours is now no longer even the top commit in your pile of fixes, so
>> now I apparently have to take that *known*incorrect* merge and fix it
>> up with an evil merge of my own.
>
> Ugh. I'm undoing my merge rather than do that evil merge that fixes up yours.
>
> So I have three choices:
>
> (a) I can just re-do your merge, and lose the two commits you had on top of it
>
> (b) I can create a new local branch with your pre-merged state, and
> cherry-pick the two commits on top of that, and then merge that, and
> then fake out the pull request.
>
> (c) I can ask you to do that fix up (rebase those two commits on top
> of the state before the broken merge), and then you can ask me to pull
> again, without the merge - same as (b) really, but I don't have to
> fake the pull request message when I create the merge.
>
> I think I'll do (c), but then probably fall back on (a) if I don't
> hear from you. (b) gets me the tree I want, but I don't like faking
> pull requests - I've occasionally pulled less than requested (exactly
> because I didn't like the top merge), but I try to avoid actually
> adding modified commits on top.
>
> Linus
next prev parent reply other threads:[~2012-03-30 19:54 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-03-30 11:38 [GIT PULL] ACPI & Power Management patches for Linux-3.4-merge Len Brown
2012-03-30 16:48 ` Rob Lee
2012-03-30 19:16 ` Linus Torvalds
2012-03-30 19:32 ` Linus Torvalds
2012-03-30 19:54 ` Len Brown [this message]
2012-03-30 21:31 ` [GIT PULL] ACPI & Power Management patches for Linux-3.4-merge - take 2 Len Brown
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4F760F6F.5060002@kernel.org \
--to=lenb@kernel.org \
--cc=daniel.lezcano@linaro.org \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=rob.lee@linaro.org \
--cc=torvalds@linux-foundation.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox