[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 1/1] MAINTAINERS: Add Python library stanza
From: |
Markus Armbruster |
Subject: |
Re: [PATCH v2 1/1] MAINTAINERS: Add Python library stanza |
Date: |
Thu, 24 Sep 2020 08:58:26 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.1 (gnu/linux) |
Philippe Mathieu-Daudé <philmd@redhat.com> writes:
> 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:.
Good point.
> Agreed. Maybe worth a comment in the section although.
Let's do it.