Philippe Mathieu-Daudé <[email protected]> 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 <[email protected]> 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 <[email protected]> >>>>> Reviewed-by: Philippe Mathieu-Daudé <[email protected]> >>>>> Tested-by: Philippe Mathieu-Daudé <[email protected]> >>>>> Reviewed-by: Alex Bennée <[email protected]> >>>>> Acked-by: Cleber Rosa <[email protected]> >>>>> Acked-by: Eduardo Habkost <[email protected]> >>>>> --- >>>>> 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 <[email protected]> >>>>> +M: Cleber Rosa <[email protected]> >>>>> +R: Eduardo Habkost <[email protected]> >>>>> +S: Maintained >>>>> +F: python/ >>>>> +T: git https://gitlab.com/jsnow/qemu.git python >>>>> + >>>>> Python scripts >>>>> M: Eduardo Habkost <[email protected]> >>>>> M: Cleber Rosa <[email protected]> >>>>> 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.
