Re: [edk2-devel] [Patch 1/1] Maintainers.txt: Update based on active community members

["Michael D Kinney" <michael.d.kinney@intel.com>]

Hi Laszlo,

I do not support orphaned categories and that option should be
removed from Maintainer.txt. One of the motivations to get 
Maintainers.txt updated is to work on the set of tasks related to 
using GitHub PRs for code review.  If a component is orphaned,
then nobody would be assigned to a PR in that area and the PR
would be stuck and would eventually be deleted for no activity.
A terrible experience for a submitter.

If there is a feature for which there is no longer any support,
then I recommend we find a way to remove it from the head of the
repository.  The feature is still available in the history and
in previous releases when it was supported.

If there is a future need for the feature and there are those that
are willing to support it, it can always be resurrected from the
history.

If it is a critical feature that will break the entire project 
if it is removed, then we must find community members that are 
willing to own it.

The immediate backup for this scenario is the EDK II Stewards, but
They may not have the background on the specific feature to maintain
it well.  For example, I am currently helping with the NetworkPkg
because there are no maintainers and I have been recruiting without
success.

I would like the see the SignedCapulePkg removed.  There are a
couple platforms in edk2-platforms that depend on it.  There is
another task to review the actively supported platforms in 
edk2-platforms.  If those platforms are removed, then SignedCapulsePkg
could be safely removed from the head of edk2.

SourceLevelDebugPkg has a similar issues of no maintainers.  The
platforms maintained in edk2 repo do not depend on it to do source
level debug.  It is more of a physical platform debug capability.
Perhaps this feature should be moved to the edk2-platform.  There
was a brief discuss at the UEFI Plugfest to update this debug 
feature because the current one depends on very old tools.

Mike

> -----Original Message-----
> From: Laszlo Ersek <lersek@redhat.com>
> Sent: Monday, October 30, 2023 4:29 AM
> To: Kinney, Michael D <michael.d.kinney@intel.com>; Yao, Jiewen
> <jiewen.yao@intel.com>; devel@edk2.groups.io; pedro.falcato@gmail.com
> Cc: Andrew Fish <afish@apple.com>; Leif Lindholm
> <quic_llindhol@quicinc.com>; Warkentin, Andrei
> <andrei.warkentin@intel.com>; West, Catharine
> <catharine.west@intel.com>; Bi, Dandan <dandan.bi@intel.com>; Daniel
> Schaefer <git@danielschaefer.me>; David Woodhouse
> <dwmw2@infradead.org>; De, Debkumar <debkumar.de@intel.com>; Dong,
> Eric <eric.dong@intel.com>; Jiang, Guomin <guomin.jiang@intel.com>;
> Wu, Hao A <hao.a.wu@intel.com>; James Bottomley <jejb@linux.ibm.com>;
> Wang, Jian J <jian.j.wang@intel.com>; Justen, Jordan L
> <jordan.l.justen@intel.com>; Julien Grall <julien@xen.org>; Peter
> Grehan <grehan@freebsd.org>; Zhang, Qi1 <qi1.zhang@intel.com>; Ng, Ray
> Han Lim <ray.han.lim.ng@intel.com>; Stefan Berger
> <stefanb@linux.ibm.com>; Hou, Wenxing <wenxing.hou@intel.com>; Lu,
> Xiaoyu1 <xiaoyu1.lu@intel.com>
> Subject: Re: [edk2-devel] [Patch 1/1] Maintainers.txt: Update based on
> active community members
> 
> On 10/30/23 06:31, Kinney, Michael D wrote:
> > There is a very good discussion here on the roles and responsibility
> > and potential suggestions for changes to the Wiki page that document
> > those roles and responsibilities.
> >
> > May I suggest that someone start a new thread that discusses
> > the proposed changes to the Wiki page and leave this thread for the
> > review of the changes to Maintainers.txt?
> 
> These are connected topics, but yes, back to "Maintainers.txt" -- do
> you
> feel that "S: Orphan" sections are acceptable in general? They're a
> first (AFAICT) for edk2's "Maintainers.txt"; we've always had the
> mechanism documented (since the creation of "Maintainers.txt", or so),
> but we never seem to have put it to use.
> 
> Laszlo



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#110345): https://edk2.groups.io/g/devel/message/110345
Mute This Topic: https://groups.io/mt/102245264/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/leave/12367111/7686176/1913456212/xyzzy [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-