Re: [Qemu-devel] MAINTAINERS leaves too many files uncovered

[Markus Armbruster]

Thomas Huth <address@hidden> writes:

> On 2018-11-21 19:14, Markus Armbruster wrote:
>> Time for another update.  My HEAD is at v3.1.0-rc2.
>
> Thanks a lot for this, Markus, very appreciated!
>
> [...]
>> Unmaintained files sorted by number of commits in the past year
>> (v2.11.0-rc2):
>> 
>> $ for i in `cat unmaintained-files`; do echo -n "$i "; git-rev-list 
>> v2.11.0-rc2.. $i | wc -l; done | awk '{ printf "%7d %s\n", $2, $1 }' | sort 
>> -rn | head -n 100
>>     139 MAINTAINERS
>>     138 configure
>>     108 tests/Makefile.include
>>      87 qemu-options.hx
>
> I still think we'd need someone to look after qemu-options.hx, since
> these are public interfaces...

QAPIfication to the rescue...

[...]
>> qdev needs a maintainer.  It has needed one since forever.
>
> Definitely. But who could do that job?

I'm afraid most (all?) candidates that have the experience lack the
bandwidth.  I certainly do.  Let's ask git:

    $ scripts/get_maintainer.pl -f hw/core/qdev.c
    get_maintainer.pl: No maintainers found, printing recent contributors.
    get_maintainer.pl: Do not blindly cc: them on patches!  Use common sense.

    Markus Armbruster <address@hidden> (commit_signer:8/14=57%)
    "Philippe Mathieu-Daudé" <address@hidden> (commit_signer:6/14=43%)
    Paolo Bonzini <address@hidden> (commit_signer:5/14=36%)
    Eric Blake <address@hidden> (commit_signer:4/14=29%)
    Laurent Vivier <address@hidden> (commit_signer:2/14=14%)
    address@hidden (open list:All patches CC here)
    $ scripts/get_maintainer.pl -f --git-blame hw/core/qdev.c
    Anthony Liguori <address@hidden> (authored 
lines:170/1114=15%,commits:49/158=31%)
    Peter Crosthwaite <address@hidden> (authored lines:149/1114=13%)
    Paolo Bonzini <address@hidden> (authored 
lines:112/1114=10%,commits:49/158=31%)
    Igor Mammedov <address@hidden> (authored lines:72/1114=6%)
    Paul Brook <address@hidden> (authored lines:65/1114=6%)
    "Andreas Färber" <address@hidden> (commits:50/158=32%)
    Markus Armbruster <address@hidden> (commits:27/158=17%)
    "Michael S. Tsirkin" <address@hidden> (commits:27/158=17%)
    address@hidden (open list:All patches CC here)

Any takers?