All of lore.kernel.org
 help / color / mirror / Atom feed
From: Laszlo Ersek <lersek@redhat.com>
To: David Woodhouse <dwmw2@infradead.org>,
	Markus Armbruster <armbru@redhat.com>
Cc: Peter Maydell <peter.maydell@linaro.org>,
	Paolo Bonzini <pbonzini@redhat.com>,
	qemu devel list <qemu-devel@nongnu.org>,
	"Jordan Justen (Intel address)" <jordan.l.justen@intel.com>,
	Ard Biesheuvel <ard.biesheuvel@linaro.org>
Subject: Re: [Qemu-devel] why restrict pull reqs to signed tags?
Date: Thu, 10 Mar 2016 13:38:59 +0100	[thread overview]
Message-ID: <56E16AE3.8090805@redhat.com> (raw)
In-Reply-To: <1457606081.124630.8.camel@infradead.org>

On 03/10/16 11:34, David Woodhouse wrote:
> On Thu, 2016-03-10 at 09:52 +0100, Markus Armbruster wrote:
>> Strawman alert: we don't *enforce* rebase.  We leave it to the
>> maintainer's discretion. Nothing stops a maintainer (or a chain of
>> them) from accepting pull requests. 
> 
> Which is all I was asking EDK2 to do. They *do* enforce rebase, which
> is wrong.
> 
> Laszlo appeared to be saying "but qemu works like this; are they wrong
> too?".
> 
> To which the answer is apparently "no, they don't work like this."
> 
> Thanks for clearing that up.

Markus, thank you for clearing that up. I failed to distinguish
"enforcement" from "practice that is applied in 99.999% of the time".

David, it doesn't change anything relative to one of my earliest emails:

http://thread.gmane.org/gmane.comp.bios.edk2.devel/8864/focus=8889

I personally agreed to your proposal very early (and have repeated that
agreement a few times since), dependent on agreement from the other edk2
maintainers too. The linear history requirement is not mine in edk2. I
don't enforce it, I comply with it. In my "unkempt" guide, I relay that
requirement, don't dictate it. My explanation of it may not have been
entirely correct, yes. However, Jordan also told you that it is
temporary, while the edk2 people's git expertise matures.

If you want to gather feedback on immediately introducing a workflow to
edk2 that allows merges, please write a focused group email to the
maintainers listed in "Maintainers.txt". Some of them might feel better
about discussing this question, and/or feel more closely addressed, if
it doesn't happen on the list.

Going forward, please refrain from over-using your "cluebat" (e.g., the
tons of bold in your email). Discussing workflow is hard enough in its
own right, you don't need to make it harder by alienating people. Linus
gets away with management by perkele because he's a chief maintainer. In
this case, it's you who wants to achieve something, even if you position
it as "fixing the workflow for everyone". You are right about merging
(and I never denied that), but I do find myself struggling harder and
harder to open your next email.

      reply	other threads:[~2016-03-10 12:39 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-03-09 10:20 [Qemu-devel] why restrict pull reqs to signed tags? Laszlo Ersek
2016-03-09 11:33 ` Paolo Bonzini
2016-03-09 11:35 ` Peter Maydell
2016-03-09 12:13   ` Laszlo Ersek
2016-03-09 12:19     ` Paolo Bonzini
2016-03-09 12:31       ` Laszlo Ersek
2016-03-09 12:33         ` Paolo Bonzini
2016-03-09 12:38         ` David Woodhouse
2016-03-09 12:40           ` Ard Biesheuvel
2016-03-09 12:44             ` Peter Maydell
2016-03-09 13:14               ` Laszlo Ersek
2016-03-09 12:34     ` David Woodhouse
2016-03-09 12:42       ` Peter Maydell
2016-03-09 13:09         ` David Woodhouse
2016-03-09 13:27           ` Peter Maydell
2016-03-09 14:13             ` David Woodhouse
2016-03-09 14:41               ` Laszlo Ersek
2016-03-10  8:21                 ` David Woodhouse
2016-03-10  8:52                   ` Markus Armbruster
2016-03-10 10:34                     ` David Woodhouse
2016-03-10 12:38                       ` Laszlo Ersek [this message]

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=56E16AE3.8090805@redhat.com \
    --to=lersek@redhat.com \
    --cc=ard.biesheuvel@linaro.org \
    --cc=armbru@redhat.com \
    --cc=dwmw2@infradead.org \
    --cc=jordan.l.justen@intel.com \
    --cc=pbonzini@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.