Re: [Qemu-devel] [RFC PATCH 46/47] MAINTAINERS: add missing entries for Coccinelle scripts

[Philippe Mathieu-Daudé]

Hi Paolo,

On 07/28/2017 08:50 AM, Paolo Bonzini wrote:
> On 28/07/2017 08:24, Fam Zheng wrote:
> > On Fri, 07/28 02:36, Philippe Mathieu-Daudé wrote:
> > > diff --git a/MAINTAINERS b/MAINTAINERS
> > > index 224890643d..7854792765 100644
> > > --- a/MAINTAINERS
> > > +++ b/MAINTAINERS
> > > @@ -1537,6 +1537,12 @@ Checkpatch
> > >   S: Odd Fixes
> > >   F: scripts/checkpatch.pl
> > >   
> > > +Coccinelle scripts
> > > +R: Philippe Mathieu-Daudé <address@hidden>
> > > +S: Orphan
> >
> > "Orphan" sounds like we don't care much anymore but I suppose this is not the
> > case. But I don't have any better idea either.
>
> Just don't add it to the file.  It doesn't have a specific owner
I thought an entry with F: but no M: was incorrect but apparently 
get_maintainer.pl works and report it as "unknown" status, as for the 
"Coroutines" section:

$ ./scripts/get_maintainer.pl -f util/coroutine-ucontext.c
Stefan Hajnoczi <address@hidden> (unknown:Coroutines)
Kevin Wolf <address@hidden> (unknown:Coroutines)
address@hidden (open list:All patches CC here)

So to enjoy the R-tag I'll remove the S-tag in v2 and see if Eric is ok 
with it :)

Regards,

Phil.

> Paolo
>
> > Fam
> >
> > > +F: scripts/cocci-macro-file.h
> > > +F: scripts/coccinelle/
> > > +
> > >   Migration
> > >   M: Juan Quintela <address@hidden>
> > >   M: Dr. David Alan Gilbert <address@hidden>
> > > @@ -1970,4 +1976,3 @@ Build system architecture
> > >   M: Daniel P. Berrange <address@hidden>
> > >   S: Odd Fixes
> > >   F: docs/devel/build-system.txt