[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 1/1] MAINTAINERS: Add Python library stanza
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH v2 1/1] MAINTAINERS: Add Python library stanza |
Date: |
Wed, 23 Sep 2020 18:49:34 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.11.0 |
On 9/23/20 5:37 PM, Eduardo Habkost wrote:
> On Wed, Sep 23, 2020 at 11:17:01AM -0400, John Snow wrote:
>> 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.
>
> Also, the actual owners for most scripts in ./scripts are already
> listed elsewhere. I believe we shouldn't really have M: lines
> for the scripts/*.py section, just R:.
Agreed. Maybe worth a comment in the section although.