[igt-dev] [RFC i-g-t 1/1] MAINTAINERS: Introduce MAINTAINERS file

[Harry Wentland]

On 2022-02-03 06:03, Petri Latvala wrote:
> On Mon, Jan 31, 2022 at 09:28:18AM -0500, Rodrigo Siqueira wrote:
>> IGT is growing in terms of new vendors actively using it. Right now, we
>> only have Intel CI to validate new patches, which can create a situation
>> where bugs get accidentally introduced to other vendors. To alleviate
>> this problem and also making easy for other developers to be aware of
>> some specific changes, I'm proposing the introduction of the MAINTAINERS
>> file as the one used in the Linux kernel. With this approach, we can
>> have multiple enginers from different companies as reviewers for
>> specific tests, and contributors should use get_maintainers script to
>> find the right people to address their changes.
>>
>> Cc: Arkadiusz Hiler <arek at hiler.eu>
>> Cc: Petri Latvala <petri.latvala at intel.com>
>> Cc: Mark Yacoub <markyacoub at google.com>
>> Cc: Jessica Zhang <quic_jesszhan at quicinc.com>
>> Cc: Abhinav Kumar <quic_abhinavk at quicinc.com>
>> Cc: Melissa Wen <mwen at igalia.com>
>> Cc: Sean Paul <seanpaul at chromium.org>
>> Cc: Harry Wentland <harry.wentland at amd.com>
>> Cc: Sun Peng Li(Leo) <sunpeng.li at amd.com>
>> Cc: Chao-kai Wang (Stylon) <stylon.wang at amd.com>
>> Cc: Wayne Lin <wayne.lin at amd.com>
>> Cc: Nicholas Choi <nicholas.choi at amd.com>
>> Cc: Martin Peres <martin.peres at mupuf.org>
>> Cc: Aurabindo Pillai <aurabindo.pillai at amd.com>
>> Cc: Bhawanpreet Lakha <bhawanpreet.lakha at amd.com>
>> Cc: Qingqing Zhuo (Lilian) <qingqing.zhuo at amd.com>
>> Cc: Solomon Chiu <solomon.chiu at amd.com>
>> Signed-off-by: Rodrigo Siqueira <Rodrigo.Siqueira at amd.com>
> 
> 
> A tentative ack on this effort from me, for getting more contact
> points formally documented.
> 
> As for the implementation of it, I know this is copied from the kernel
> so I know the reason why they're there, but reduce the useless
> complexity in the documentation comment like the type of version
> control in a subtree, and things that just aren't true for IGT, like
> asking for testing from 4 or 5 people.
> 

I tend to agree with the statement that this has too much useless
complexity and would love if that'd be dropped/cleaned up, though
that could come at a later time (and might be less important if
IGT moves to gitlab MRs).

Harry

> (Not to mention you're using S to describe the people, whereas the
> original usage in kernel is to describe the files)
> 
> And the scripting to use this should be contained within
> IGT. get_maintainer.pl is GPL-2.0 so we could in theory just copy
> it. It isn't used to produce binaries so IGT should still remain
> distributable with MIT terms but IANAL...
> 
>