* [Qemu-devel] [PATCH] scripts/get_maintainer.pl: allow "odd fixes"
@ 2014-10-21 11:35 Michael S. Tsirkin
2014-10-21 16:55 ` Don Slutz
2014-10-22 7:22 ` Michael S. Tsirkin
0 siblings, 2 replies; 11+ messages in thread
From: Michael S. Tsirkin @ 2014-10-21 11:35 UTC (permalink / raw)
To: qemu-devel; +Cc: Paolo Bonzini, Gerd Hoffmann, Markus Armbruster, Peter Maydell
We have a bunch of modules in "Odd fixes"
status, scripts/get_maintainer.pl ignores that.
Reported-by: Gerd Hoffmann <kraxel@redhat.com>
Cc: Paolo Bonzini <pbonzini@redhat.com>
Cc: Peter Maydell <peter.maydell@linaro.org>
Cc: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
---
scripts/get_maintainer.pl | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/scripts/get_maintainer.pl b/scripts/get_maintainer.pl
index 38334de..d5eee8c 100755
--- a/scripts/get_maintainer.pl
+++ b/scripts/get_maintainer.pl
@@ -514,7 +514,7 @@ sub range_is_maintained {
my $type = $1;
my $value = $2;
if ($type eq 'S') {
- if ($value =~ /(maintain|support)/i) {
+ if ($value =~ /(maintain|support|fix)/i) {
return 1;
}
}
--
MST
^ permalink raw reply related [flat|nested] 11+ messages in thread* Re: [Qemu-devel] [PATCH] scripts/get_maintainer.pl: allow "odd fixes"
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
1 sibling, 0 replies; 11+ messages in thread
From: Don Slutz @ 2014-10-21 16:55 UTC (permalink / raw)
To: Michael S. Tsirkin, qemu-devel
Cc: Peter Maydell, Paolo Bonzini, Gerd Hoffmann, Markus Armbruster
On 10/21/14 07:35, Michael S. Tsirkin wrote:
> We have a bunch of modules in "Odd fixes"
> status, scripts/get_maintainer.pl ignores that.
>
> Reported-by: Gerd Hoffmann <kraxel@redhat.com>
> Cc: Paolo Bonzini <pbonzini@redhat.com>
> Cc: Peter Maydell <peter.maydell@linaro.org>
> Cc: Markus Armbruster <armbru@redhat.com>
> Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
> ---
> scripts/get_maintainer.pl | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/scripts/get_maintainer.pl b/scripts/get_maintainer.pl
> index 38334de..d5eee8c 100755
> --- a/scripts/get_maintainer.pl
> +++ b/scripts/get_maintainer.pl
> @@ -514,7 +514,7 @@ sub range_is_maintained {
> my $type = $1;
> my $value = $2;
> if ($type eq 'S') {
> - if ($value =~ /(maintain|support)/i) {
> + if ($value =~ /(maintain|support|fix)/i) {
> return 1;
> }
> }
Looks good to me.
Reviewed-by: Don Slutz <dslutz@verizon.com>
-Don Slutz
^ permalink raw reply [flat|nested] 11+ messages in thread* Re: [Qemu-devel] [PATCH] scripts/get_maintainer.pl: allow "odd fixes"
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 7:47 ` Markus Armbruster
1 sibling, 2 replies; 11+ messages in thread
From: Michael S. Tsirkin @ 2014-10-22 7:22 UTC (permalink / raw)
To: qemu-devel; +Cc: Paolo Bonzini, Gerd Hoffmann, Markus Armbruster, Peter Maydell
On Tue, Oct 21, 2014 at 02:35:09PM +0300, Michael S. Tsirkin wrote:
> We have a bunch of modules in "Odd fixes"
> status, scripts/get_maintainer.pl ignores that.
>
> Reported-by: Gerd Hoffmann <kraxel@redhat.com>
> Cc: Paolo Bonzini <pbonzini@redhat.com>
> Cc: Peter Maydell <peter.maydell@linaro.org>
> Cc: Markus Armbruster <armbru@redhat.com>
> Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
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?
> ---
> scripts/get_maintainer.pl | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/scripts/get_maintainer.pl b/scripts/get_maintainer.pl
> index 38334de..d5eee8c 100755
> --- a/scripts/get_maintainer.pl
> +++ b/scripts/get_maintainer.pl
> @@ -514,7 +514,7 @@ sub range_is_maintained {
> my $type = $1;
> my $value = $2;
> if ($type eq 'S') {
> - if ($value =~ /(maintain|support)/i) {
> + if ($value =~ /(maintain|support|fix)/i) {
> return 1;
> }
> }
> --
> MST
^ permalink raw reply [flat|nested] 11+ messages in thread* Re: [Qemu-devel] [PATCH] scripts/get_maintainer.pl: allow "odd fixes"
2014-10-22 7:22 ` Michael S. Tsirkin
@ 2014-10-22 7:45 ` Paolo Bonzini
2014-10-22 8:08 ` Michael S. Tsirkin
2014-10-22 7:47 ` Markus Armbruster
1 sibling, 1 reply; 11+ messages in thread
From: Paolo Bonzini @ 2014-10-22 7:45 UTC (permalink / raw)
To: Michael S. Tsirkin, qemu-devel
Cc: Peter Maydell, Gerd Hoffmann, Markus Armbruster
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. And unless
they're trivial or egregious, I would not spend much time on bug
reports. It just sets expectations right.
Paolo
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [Qemu-devel] [PATCH] scripts/get_maintainer.pl: allow "odd fixes"
2014-10-22 7:45 ` Paolo Bonzini
@ 2014-10-22 8:08 ` Michael S. Tsirkin
2014-10-22 8:07 ` Paolo Bonzini
0 siblings, 1 reply; 11+ messages in thread
From: Michael S. Tsirkin @ 2014-10-22 8:08 UTC (permalink / raw)
To: Paolo Bonzini; +Cc: Peter Maydell, qemu-devel, Markus Armbruster, Gerd Hoffmann
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
^ permalink raw reply [flat|nested] 11+ messages in thread* Re: [Qemu-devel] [PATCH] scripts/get_maintainer.pl: allow "odd fixes"
2014-10-22 8:08 ` Michael S. Tsirkin
@ 2014-10-22 8:07 ` Paolo Bonzini
2014-10-22 8:13 ` Michael S. Tsirkin
0 siblings, 1 reply; 11+ messages in thread
From: Paolo Bonzini @ 2014-10-22 8:07 UTC (permalink / raw)
To: Michael S. Tsirkin
Cc: Peter Maydell, qemu-devel, Markus Armbruster, Gerd Hoffmann
On 10/22/2014 10:08 AM, Michael S. Tsirkin wrote:
>> >
>> > 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.
To me it means that I have time to review patches, but not to do my own
work on it.
Paolo
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [Qemu-devel] [PATCH] scripts/get_maintainer.pl: allow "odd fixes"
2014-10-22 8:07 ` Paolo Bonzini
@ 2014-10-22 8:13 ` Michael S. Tsirkin
2014-10-22 8:18 ` Paolo Bonzini
0 siblings, 1 reply; 11+ messages in thread
From: Michael S. Tsirkin @ 2014-10-22 8:13 UTC (permalink / raw)
To: Paolo Bonzini; +Cc: Peter Maydell, qemu-devel, Markus Armbruster, Gerd Hoffmann
On Wed, Oct 22, 2014 at 10:07:52AM +0200, Paolo Bonzini wrote:
>
>
> On 10/22/2014 10:08 AM, Michael S. Tsirkin wrote:
> >> >
> >> > 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.
>
> To me it means that I have time to review patches, but not to do my own
> work on it.
>
> Paolo
Intersting.
review is "other than throw the odd patch in" so it's hardly
the literal meaning of this text.
--
MST
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [Qemu-devel] [PATCH] scripts/get_maintainer.pl: allow "odd fixes"
2014-10-22 8:13 ` Michael S. Tsirkin
@ 2014-10-22 8:18 ` Paolo Bonzini
0 siblings, 0 replies; 11+ messages in thread
From: Paolo Bonzini @ 2014-10-22 8:18 UTC (permalink / raw)
To: Michael S. Tsirkin
Cc: Peter Maydell, qemu-devel, Markus Armbruster, Gerd Hoffmann
On 10/22/2014 10:13 AM, Michael S. Tsirkin wrote:
> > To me it means that I have time to review patches, but not to do my own
> > work on it.
>
> Intersting.
> review is "other than throw the odd patch in" so it's hardly
> the literal meaning of this text.
Well, review is just the prerequisite/necessary evil for throwing the
odd patch in. :)
Paolo
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [Qemu-devel] [PATCH] scripts/get_maintainer.pl: allow "odd fixes"
2014-10-22 7:22 ` Michael S. Tsirkin
2014-10-22 7:45 ` Paolo Bonzini
@ 2014-10-22 7:47 ` Markus Armbruster
2014-10-22 7:58 ` Paolo Bonzini
1 sibling, 1 reply; 11+ messages in thread
From: Markus Armbruster @ 2014-10-22 7:47 UTC (permalink / raw)
To: Michael S. Tsirkin
Cc: Peter Maydell, Paolo Bonzini, qemu-devel, Gerd Hoffmann
"Michael S. Tsirkin" <mst@redhat.com> writes:
> On Tue, Oct 21, 2014 at 02:35:09PM +0300, Michael S. Tsirkin wrote:
>> We have a bunch of modules in "Odd fixes"
>> status, scripts/get_maintainer.pl ignores that.
>>
>> Reported-by: Gerd Hoffmann <kraxel@redhat.com>
>> Cc: Paolo Bonzini <pbonzini@redhat.com>
>> Cc: Peter Maydell <peter.maydell@linaro.org>
>> Cc: Markus Armbruster <armbru@redhat.com>
>> Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
>
> Actually, it does not ignore that.
> What it does it list status as "odd fixer".
Like this:
$ scripts/get_maintainer.pl -f --no-git-fallback hw/ide/core.c
Kevin Wolf <kwolf@redhat.com> (odd fixer:IDE)
Stefan Hajnoczi <stefanha@redhat.com> (odd fixer:IDE)
What's wrong with that?
> 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?
What exactly does this patch fix?
^ permalink raw reply [flat|nested] 11+ messages in thread* Re: [Qemu-devel] [PATCH] scripts/get_maintainer.pl: allow "odd fixes"
2014-10-22 7:47 ` Markus Armbruster
@ 2014-10-22 7:58 ` Paolo Bonzini
2014-10-22 8:07 ` Markus Armbruster
0 siblings, 1 reply; 11+ messages in thread
From: Paolo Bonzini @ 2014-10-22 7:58 UTC (permalink / raw)
To: Markus Armbruster, Michael S. Tsirkin
Cc: Peter Maydell, qemu-devel, Gerd Hoffmann
On 10/22/2014 09:47 AM, Markus Armbruster wrote:
> Like this:
>
> $ scripts/get_maintainer.pl -f --no-git-fallback hw/ide/core.c
> Kevin Wolf <kwolf@redhat.com> (odd fixer:IDE)
> Stefan Hajnoczi <stefanha@redhat.com> (odd fixer:IDE)
>
> What's wrong with that?
Michael's patch fixes the case where you do _not_ have --no-git-fallback:
$ scripts/get_maintainer.pl -f hw/ide/core.c
Kevin Wolf <kwolf@redhat.com> (odd fixer:IDE,commit_signer:6/27=22%)
Stefan Hajnoczi <stefanha@redhat.com> (odd fixer:IDE,commit_signer:18/27=67%)
John Snow <jsnow@redhat.com> (commit_signer:13/27=48%)
Paolo Bonzini <pbonzini@redhat.com> (commit_signer:11/27=41%)
Fam Zheng <famz@redhat.com> (commit_signer:4/27=15%)
With the patch:
$ scripts/get_maintainer.pl -f hw/ide/core.c
Kevin Wolf <kwolf@redhat.com> (odd fixer:IDE)
Stefan Hajnoczi <stefanha@redhat.com> (odd fixer:IDE)
which behaves the same as:
$ scripts/get_maintainer.pl -f block/qed.c
Kevin Wolf <kwolf@redhat.com> (supporter:Block)
Stefan Hajnoczi <stefanha@redhat.com> (supporter:Block)
Paolo
^ permalink raw reply [flat|nested] 11+ messages in thread* Re: [Qemu-devel] [PATCH] scripts/get_maintainer.pl: allow "odd fixes"
2014-10-22 7:58 ` Paolo Bonzini
@ 2014-10-22 8:07 ` Markus Armbruster
0 siblings, 0 replies; 11+ messages in thread
From: Markus Armbruster @ 2014-10-22 8:07 UTC (permalink / raw)
To: Paolo Bonzini
Cc: Peter Maydell, Gerd Hoffmann, qemu-devel, Michael S. Tsirkin
Paolo Bonzini <pbonzini@redhat.com> writes:
> On 10/22/2014 09:47 AM, Markus Armbruster wrote:
>> Like this:
>>
>> $ scripts/get_maintainer.pl -f --no-git-fallback hw/ide/core.c
>> Kevin Wolf <kwolf@redhat.com> (odd fixer:IDE)
>> Stefan Hajnoczi <stefanha@redhat.com> (odd fixer:IDE)
>>
>> What's wrong with that?
>
> Michael's patch fixes the case where you do _not_ have --no-git-fallback:
>
> $ scripts/get_maintainer.pl -f hw/ide/core.c
> Kevin Wolf <kwolf@redhat.com> (odd fixer:IDE,commit_signer:6/27=22%)
> Stefan Hajnoczi <stefanha@redhat.com> (odd
> fixer:IDE,commit_signer:18/27=67%)
> John Snow <jsnow@redhat.com> (commit_signer:13/27=48%)
> Paolo Bonzini <pbonzini@redhat.com> (commit_signer:11/27=41%)
> Fam Zheng <famz@redhat.com> (commit_signer:4/27=15%)
>
> With the patch:
>
> $ scripts/get_maintainer.pl -f hw/ide/core.c
> Kevin Wolf <kwolf@redhat.com> (odd fixer:IDE)
> Stefan Hajnoczi <stefanha@redhat.com> (odd fixer:IDE)
>
> which behaves the same as:
>
> $ scripts/get_maintainer.pl -f block/qed.c
> Kevin Wolf <kwolf@redhat.com> (supporter:Block)
> Stefan Hajnoczi <stefanha@redhat.com> (supporter:Block)
Got it, thanks!
Shows how automatic --no-git-fallback has become for me...
>> > 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?
>>
>> What exactly does this patch fix?
No objections to applying the patch.
^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2014-10-22 8:18 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
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
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
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).