diff mbox

[PATCHv2,2/2] MAINTAINERS: remove Blue Swirl as SPARC maintainer

Message ID 1465849380-29388-3-git-send-email-mark.cave-ayland@ilande.co.uk
State New
Headers show

Commit Message

Mark Cave-Ayland June 13, 2016, 8:23 p.m. UTC 
Blue is no longer active in the QEMU project, so remove him from the list of
SPARC maintainers.

Signed-off-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
CC: Blue Swirl <blauwirbel@gmail.com>
---
 MAINTAINERS |    1 -
 1 file changed, 1 deletion(-)

Comments

Markus Armbruster June 14, 2016, 8:33 a.m. UTC | #1 
Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk> writes:

> Blue is no longer active in the QEMU project, so remove him from the list of
> SPARC maintainers.
>
> Signed-off-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
> CC: Blue Swirl <blauwirbel@gmail.com>
> ---
>  MAINTAINERS |    1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 081bf20..fdd0057 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -189,7 +189,6 @@ F: hw/sh4/
>  F: disas/sh4.c
>  
>  SPARC
> -M: Blue Swirl <blauwirbel@gmail.com>
>  M: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
>  M: Artyom Tarasenko <atar4qemu@gmail.com>
>  S: Maintained

What about the other things Blue nominally maintains?

    SPARC Machines
    --------------
    Sun4m
    M: Blue Swirl <blauwirbel@gmail.com>
    M: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
    S: Maintained
    F: hw/sparc/sun4m.c

    Sun4u
    M: Blue Swirl <blauwirbel@gmail.com>
    M: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
    S: Maintained
    F: hw/sparc64/sun4u.c

These two still have a maintainer (you).  The remaining ones are
effectively orphaned, but you can't see that in MAINTAINERS.  I feel
keeping status there honest is important.

    Subsystems
    ----------
    Checkpatch
    M: Blue Swirl <blauwirbel@gmail.com>
    S: Odd Fixes
    F: scripts/checkpatch.pl

get_maintainer.pl --no-m --git:
  Paolo Bonzini <pbonzini@redhat.com> (commit_signer:9/12=75%)
  Leonid Bloch <leonid@daynix.com> (commit_signer:2/12=17%)
  Michael Tokarev <mjt@tls.msk.ru> (commit_signer:2/12=17%)
  Peter Maydell <peter.maydell@linaro.org> (commit_signer:2/12=17%)
  Stefan Weil <sw@weilnetz.de> (commit_signer:2/12=17%)
--git-blame:
  Blue Swirl <blauwirbel@gmail.com> (authored lines:2374/2607=91%,commits:11/28=39%)
  Paolo Bonzini <pbonzini@redhat.com> (commits:10/28=36%)
  Stefan Hajnoczi <stefanha@redhat.com> (commits:5/28=18%)
  Don Slutz <Don@CloudSwitch.com> (commits:4/28=14%)
  Stefan Weil <sw@weilnetz.de> (commits:3/28=11%)
  qemu-devel@nongnu.org (open list:All patches CC here)

    Usermode Emulation
    ------------------
    BSD user
    M: Blue Swirl <blauwirbel@gmail.com>
    S: Maintained
    F: bsd-user/

    Tiny Code Generator (TCG)
    -------------------------
    SPARC target
    M: Blue Swirl <blauwirbel@gmail.com>
    S: Maintained
    F: tcg/sparc/
    F: disas/sparc.c
Paolo Bonzini June 14, 2016, 9:09 a.m. UTC | #2 
On 14/06/2016 10:33, Markus Armbruster wrote:
> The remaining ones are
> effectively orphaned, but you can't see that in MAINTAINERS.  I feel
> keeping status there honest is important.
> 
>     Subsystems
>     ----------
>     Checkpatch
>     M: Blue Swirl <blauwirbel@gmail.com>
>     S: Odd Fixes
>     F: scripts/checkpatch.pl

This one is okay to remove M and even bump to Maintained.  It is in
obviously heavy use and has been in the "miscellaneous" pool for a
while.  But I don't really long for having my name attached to it. :)

> 
>     Usermode Emulation
>     ------------------
>     BSD user
>     M: Blue Swirl <blauwirbel@gmail.com>
>     S: Maintained
>     F: bsd-user/

The FreeBSD guys have effectively forked bsd-user.  Orphan is appropriate.

>     Tiny Code Generator (TCG)
>     -------------------------
>     SPARC target
>     M: Blue Swirl <blauwirbel@gmail.com>
>     S: Maintained
>     F: tcg/sparc/
>     F: disas/sparc.c

Remove M and leave it as Odd Fixes, the TCG maintainers can have a look
at the occasional patch.

Paolo
Mark Cave-Ayland June 14, 2016, 7 p.m. UTC | #3 
On 14/06/16 09:33, Markus Armbruster wrote:

> Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk> writes:
> 
>> Blue is no longer active in the QEMU project, so remove him from the list of
>> SPARC maintainers.
>>
>> Signed-off-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
>> CC: Blue Swirl <blauwirbel@gmail.com>
>> ---
>>  MAINTAINERS |    1 -
>>  1 file changed, 1 deletion(-)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 081bf20..fdd0057 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -189,7 +189,6 @@ F: hw/sh4/
>>  F: disas/sh4.c
>>  
>>  SPARC
>> -M: Blue Swirl <blauwirbel@gmail.com>
>>  M: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
>>  M: Artyom Tarasenko <atar4qemu@gmail.com>
>>  S: Maintained
> 
> What about the other things Blue nominally maintains?
> 
>     SPARC Machines
>     --------------
>     Sun4m
>     M: Blue Swirl <blauwirbel@gmail.com>
>     M: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
>     S: Maintained
>     F: hw/sparc/sun4m.c
> 
>     Sun4u
>     M: Blue Swirl <blauwirbel@gmail.com>
>     M: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
>     S: Maintained
>     F: hw/sparc64/sun4u.c
> 
> These two still have a maintainer (you).  The remaining ones are
> effectively orphaned, but you can't see that in MAINTAINERS.  I feel
> keeping status there honest is important.

Looks like I missed these. I've spoken with Artyom about this off-list
and we're both willing to help out with sun4m/sun4u - I'll update the
patch to add ourselves, remove Blue and resubmit.

>     Subsystems
>     ----------
>     Checkpatch
>     M: Blue Swirl <blauwirbel@gmail.com>
>     S: Odd Fixes
>     F: scripts/checkpatch.pl
> 
> get_maintainer.pl --no-m --git:
>   Paolo Bonzini <pbonzini@redhat.com> (commit_signer:9/12=75%)
>   Leonid Bloch <leonid@daynix.com> (commit_signer:2/12=17%)
>   Michael Tokarev <mjt@tls.msk.ru> (commit_signer:2/12=17%)
>   Peter Maydell <peter.maydell@linaro.org> (commit_signer:2/12=17%)
>   Stefan Weil <sw@weilnetz.de> (commit_signer:2/12=17%)
> --git-blame:
>   Blue Swirl <blauwirbel@gmail.com> (authored lines:2374/2607=91%,commits:11/28=39%)
>   Paolo Bonzini <pbonzini@redhat.com> (commits:10/28=36%)
>   Stefan Hajnoczi <stefanha@redhat.com> (commits:5/28=18%)
>   Don Slutz <Don@CloudSwitch.com> (commits:4/28=14%)
>   Stefan Weil <sw@weilnetz.de> (commits:3/28=11%)
>   qemu-devel@nongnu.org (open list:All patches CC here)
> 
>     Usermode Emulation
>     ------------------
>     BSD user
>     M: Blue Swirl <blauwirbel@gmail.com>
>     S: Maintained
>     F: bsd-user/
> 
>     Tiny Code Generator (TCG)
>     -------------------------
>     SPARC target
>     M: Blue Swirl <blauwirbel@gmail.com>
>     S: Maintained
>     F: tcg/sparc/
>     F: disas/sparc.c

Sadly I can't help with TCG SPARC target as I don't own any SPARC
hardware (ironically the reason I started using QEMU in the first place..).

Outside of SPARC, I don't feel that it's my role to remove Blue from the
other areas of MAINTAINERS; perhaps that would best come from Peter?


ATB,

Mark.
diff mbox

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index 081bf20..fdd0057 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -189,7 +189,6 @@  F: hw/sh4/
 F: disas/sh4.c
 
 SPARC
-M: Blue Swirl <blauwirbel@gmail.com>
 M: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
 M: Artyom Tarasenko <atar4qemu@gmail.com>
 S: Maintained