[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: |
Markus Armbruster |
Subject: |
Re: [Qemu-trivial] [Qemu-devel] [PATCH for-3.1 25/25] MAINTAINERS: Remove duplicate entries of address@hidden |
Date: |
Tue, 27 Nov 2018 09:59:24 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux) |
Thomas Huth <address@hidden> writes:
> On 2018-11-25 21:50, Philippe Mathieu-Daudé wrote:
>> The list is always selected by the 'All patches CC here' section.
>>
>> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
>> ---
>> MAINTAINERS | 13 -------------
>> 1 file changed, 13 deletions(-)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index c616861ca3..9d6dae71ff 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -109,7 +109,6 @@ L: address@hidden
>> Guest CPU cores (TCG):
>> ----------------------
>> Overall
>> -L: address@hidden
>> M: Peter Crosthwaite <address@hidden>
>> M: Richard Henderson <address@hidden>
>> R: Paolo Bonzini <address@hidden>
>> @@ -403,30 +402,25 @@ Hosts:
>> ------
>>
>> LINUX
>> -L: address@hidden
>> S: Maintained
>> F: linux-*
>> F: linux-headers/
>>
>> 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.
- Re: [Qemu-trivial] [PATCH for-3.1 21/25] MAINTAINERS: Add a missing entry for the NVDIMM device, (continued)