[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [Qemu-devel] [PATCH for-3.1 25/25] MAINTAINERS: Remov
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-trivial] [Qemu-devel] [PATCH for-3.1 25/25] MAINTAINERS: Remove duplicate entries of address@hidden |
Date: |
Tue, 27 Nov 2018 10:20:34 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.3.0 |
On 27/11/18 10:18, Paolo Bonzini wrote:
> On 27/11/18 09:59, Markus Armbruster wrote:
>>>> LINUX
>>>> -L: address@hidden
>>>> S: Maintained
>>>> F: linux-*
>
> This pattern only matches
>
> ./block/linux-aio.c
> ./stubs/linux-aio.c
>
> so it can be removed
>
OK
>>>> F: linux-headers/
>
> For this I propose mst, Conny and myself.
Or remove... I'll let them opinate.
>
>>>> POSIX
>>>> -L: address@hidden
>>>> S: Maintained
>>>> F: *posix*
>>>
>>> I agree that we can remove the "L: qemu-devel" entries everywhere, but
>>> for these subsystems that have no "M:" entry, but "S: Maintained", this
>>> now looks somewhat strange - the subsystem is "Maintained" but has no
>>> maintainer? I think we need somebody who feels responsible to pick up
>>> patches here...?
>>
>> It looks strange because it *is* strange. If we take the definition of
>> S: in MAINTAINERS seriously, we need to appoint a maintainer (a person,
>> not a mailing list), or downgrade to S: Orphan.
>
> I can volunteer for this one too, but please change the F: patterns to
>
> F: os-posix.c
> F: include/sysemu/os-posix.h
> F: util/*posix*.c
> F: include/qemu/*posix*.h
OK, will do.
Thanks Paolo,
Phil.
>
> Thanks,
>
> Paolo
>
- Re: [Qemu-trivial] [PATCH for-3.1 22/25] MAINTAINERS: Add a missing entry to the New World machines, (continued)