Re: [PATCH v2 1/1] MAINTAINERS: Add Python library stanza

[John Snow <jsnow@redhat.com>]

On 9/23/20 11:14 AM, Markus Armbruster wrote:
> John Snow <jsnow@redhat.com> writes:
> 
>> I'm proposing that I split the actual Python library off from the other
>> miscellaneous python scripts we have and declare it maintained. Add
>> myself as a maintainer of this folder, along with Cleber.
>>
>> v2: change python/* to python/, thanks Alex.
>>
>> Signed-off-by: John Snow <jsnow@redhat.com>
>> Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
>> Tested-by: Philippe Mathieu-Daudé <philmd@redhat.com>
>> Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
>> Acked-by: Cleber Rosa <crosa@redhat.com>
>> Acked-by: Eduardo Habkost <ehabkost@redhat.com>
>> ---
>>   MAINTAINERS | 9 ++++++++-
>>   1 file changed, 8 insertions(+), 1 deletion(-)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 3d17cad19a..c0222ee645 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -2355,11 +2355,18 @@ S: Maintained
>>   F: include/sysemu/cryptodev*.h
>>   F: backends/cryptodev*.c
>>   
>> +Python library
>> +M: John Snow <jsnow@redhat.com>
>> +M: Cleber Rosa <crosa@redhat.com>
>> +R: Eduardo Habkost <ehabkost@redhat.com>
>> +S: Maintained
>> +F: python/
>> +T: git https://gitlab.com/jsnow/qemu.git python
>> +
>>   Python scripts
>>   M: Eduardo Habkost <ehabkost@redhat.com>
>>   M: Cleber Rosa <crosa@redhat.com>
>>   S: Odd fixes
>> -F: python/qemu/*py
>>   F: scripts/*.py
>>   F: tests/*.py
> 
> Separate sections just so you can have the appropriate S:, or is there a
> deeper logic behind the split?
> 

Yes, different intended levels of support. I don't currently have the 
bandwidth to promise support for the miscellaneous Python scripts in 
./scripts, but I do promise to support to a higher level the code in 
./python.

Over time, I intend to migrate things into ./python, but there are some 
series that need to happen first before I start doing that.

--js