qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: "Michael S. Tsirkin" <mst@redhat.com>
To: Paolo Bonzini <pbonzini@redhat.com>
Cc: Peter Maydell <peter.maydell@linaro.org>,
	qemu-devel@nongnu.org, Markus Armbruster <armbru@redhat.com>,
	Gerd Hoffmann <kraxel@redhat.com>
Subject: Re: [Qemu-devel] [PATCH] scripts/get_maintainer.pl: allow "odd fixes"
Date: Wed, 22 Oct 2014 11:08:44 +0300	[thread overview]
Message-ID: <20141022080844.GA4789@redhat.com> (raw)
In-Reply-To: <54476089.7020402@redhat.com>

On Wed, Oct 22, 2014 at 09:45:13AM +0200, Paolo Bonzini wrote:
> On 10/22/2014 09:22 AM, Michael S. Tsirkin wrote:
> > Actually, it does not ignore that.
> > What it does it list status as "odd fixer".
> > 
> > So this boils down to the fact that we use
> > "odd fixer" incorrectly.
> > 
> > I will apply this for now but maybe we should
> > change MAINTAINERS switching all "Odd fixes" to
> > "Maintained" and then revert this patch.
> > 
> > Thoughts?
> 
> To me "odd fixes" means I may help shepherding your patches into the
> tree, but I would rather revert a buggy patch than fix it.

OK but this is not what MAINTAINERS says:


           Odd Fixes:   It has a maintainer but they don't have time to do
                        much other than throw the odd patch in. See below.


I don't know what "See below" refers to.
But it does seem to imply maintainers don't have time to
review patches, so at some level, enabling fallback in
this case seems to make sense.



> they're trivial or egregious, I would not spend much time on bug
> reports.  It just sets expectations right.
> 
> Paolo

  reply	other threads:[~2014-10-22  8:05 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-10-21 11:35 [Qemu-devel] [PATCH] scripts/get_maintainer.pl: allow "odd fixes" Michael S. Tsirkin
2014-10-21 16:55 ` Don Slutz
2014-10-22  7:22 ` Michael S. Tsirkin
2014-10-22  7:45   ` Paolo Bonzini
2014-10-22  8:08     ` Michael S. Tsirkin [this message]
2014-10-22  8:07       ` Paolo Bonzini
2014-10-22  8:13         ` Michael S. Tsirkin
2014-10-22  8:18           ` Paolo Bonzini
2014-10-22  7:47   ` Markus Armbruster
2014-10-22  7:58     ` Paolo Bonzini
2014-10-22  8:07       ` Markus Armbruster

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=20141022080844.GA4789@redhat.com \
    --to=mst@redhat.com \
    --cc=armbru@redhat.com \
    --cc=kraxel@redhat.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).