From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by spool.mail.gandi.net (Postfix) with ESMTPS id 5F320AC115F for ; Tue, 31 Oct 2023 10:17:00 +0000 (UTC) DKIM-Signature: a=rsa-sha256; bh=KiQYs6J/E1Sr2hU7OuvtJ/3qdH6Ki+FYnJs/u2DeHpM=; c=relaxed/simple; d=groups.io; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From:In-Reply-To:Precedence:List-Subscribe:List-Help:Sender:List-Id:Mailing-List:Delivered-To:Reply-To:List-Unsubscribe-Post:List-Unsubscribe:Content-Language:Content-Type:Content-Transfer-Encoding; s=20140610; t=1698747419; v=1; b=c2cAVLTSuFKGObmhscVVMh0R5Y2p2W0WBS8IuGrVODHp0Zj+OY0m58HHzA+pkACcVQx7IYRu GHi2ZE30PxkugsXtfhQFmVnJf8AGUd6eoT2mzJNqNDrQ3fw1v2kEVwRKgFDrkEcmFRMgXZ0gSQb 25JHg0CjO09ueAcNgQUSjlTQ= X-Received: by 127.0.0.2 with SMTP id keFdYY7687511x16bnGo4neg; Tue, 31 Oct 2023 03:16:59 -0700 X-Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by mx.groups.io with SMTP id smtpd.web10.182643.1698747418364306923 for ; Tue, 31 Oct 2023 03:16:58 -0700 X-Received: from mimecast-mx02.redhat.com (mimecast-mx02.redhat.com [66.187.233.88]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-659-bvEuFBCZPeWapnoRw-SPQg-1; Tue, 31 Oct 2023 06:16:48 -0400 X-MC-Unique: bvEuFBCZPeWapnoRw-SPQg-1 X-Received: from smtp.corp.redhat.com (int-mx09.intmail.prod.int.rdu2.redhat.com [10.11.54.9]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id ECC37828AC4; Tue, 31 Oct 2023 10:16:46 +0000 (UTC) X-Received: from [10.39.195.34] (unknown [10.39.195.34]) by smtp.corp.redhat.com (Postfix) with ESMTPS id CA3D0492BE0; Tue, 31 Oct 2023 10:16:42 +0000 (UTC) Message-ID: <9a6c482f-0188-0c44-0446-ffb106a67173@redhat.com> Date: Tue, 31 Oct 2023 11:16:41 +0100 MIME-Version: 1.0 Subject: Re: [edk2-devel] [Patch 1/1] Maintainers.txt: Update based on active community members To: "Kinney, Michael D" , "Yao, Jiewen" , "devel@edk2.groups.io" , "pedro.falcato@gmail.com" Cc: Andrew Fish , Leif Lindholm , "Warkentin, Andrei" , "West, Catharine" , "Bi, Dandan" , Daniel Schaefer , David Woodhouse , "De, Debkumar" , "Dong, Eric" , "Jiang, Guomin" , "Wu, Hao A" , James Bottomley , "Wang, Jian J" , "Justen, Jordan L" , Julien Grall , Peter Grehan , "Zhang, Qi1" , "Ng, Ray Han Lim" , Stefan Berger , "Hou, Wenxing" , "Lu, Xiaoyu1" References: <20231028192330.1031-1-michael.d.kinney@intel.com> <17ced86a-cbdd-e9b6-3011-4185840c0ab9@redhat.com> From: "Laszlo Ersek" In-Reply-To: X-Scanned-By: MIMEDefang 3.4.1 on 10.11.54.9 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Precedence: Bulk List-Subscribe: List-Help: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Reply-To: devel@edk2.groups.io,lersek@redhat.com List-Unsubscribe-Post: List-Unsubscribe=One-Click List-Unsubscribe: X-Gm-Message-State: ZcljiIOtw9BTXhcaeVtex6bGx7686176AA= Content-Language: en-US Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-GND-Status: LEGIT Authentication-Results: spool.mail.gandi.net; dkim=pass header.d=groups.io header.s=20140610 header.b=c2cAVLTS; dmarc=fail reason="SPF not aligned (relaxed), DKIM not aligned (relaxed)" header.from=redhat.com (policy=none); spf=pass (spool.mail.gandi.net: domain of bounce@groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce@groups.io On 10/30/23 23:18, Kinney, Michael D wrote: > Hi Laszlo, >=20 > I do not support orphaned categories and that option should be > removed from Maintainer.txt. One of the motivations to get=20 > Maintainers.txt updated is to work on the set of tasks related to=20 > using GitHub PRs for code review. I see. I didn't know. So, the mailing list based review process is going away. > 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. I agree, although just because a PR is auto-assigned to reviewers, tehre is no guarantee that those reviewers will provide timely feedback. (The current, long response times on the list may have two reasons; one, reviewers missing patches (in spite of the direct CC's); two, reviewers not acting on the patches they are aware of. Reviewing PRs on GitHub may help with the former, but I'm doubtful it will help with the latter. Anyway, I'm not trying to object to the workflow change.) > 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. OK! > 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. >=20 > If it is a critical feature that will break the entire project=20 > if it is removed, then we must find community members that are=20 > willing to own it. >=20 > 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. It's unfortunate that NetworkPkg has no dedicated maintainer; UEFI network boot (for example in OVMF guests) is certainly used in the industry= . I'll try to help out with patch reviews for NetworkPkg as well. > 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=20 > edk2-platforms. If those platforms are removed, then SignedCapulsePkg > could be safely removed from the head of edk2. ... or else SignedCapulsePkg could be moved to edk2-platforms. (While I prefer to keep everything in one big tree, I agree that moving SignedCapulsePkg to edk2-platforms would be consistent with past subsystem movements.) > 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=20 > feature because the current one depends on very old tools. I'd even welcome that, as I see SOURCE_DEBUG_ENABLE an unnecessary (not functional) complication in the OVMF DSC files. OK, let us review the patch again from scratch, with these points in mind. Thanks Laszlo -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#110394): https://edk2.groups.io/g/devel/message/110394 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/19134562= 12/xyzzy [rebecca@openfw.io] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-