From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=209.132.183.28; helo=mx1.redhat.com; envelope-from=lersek@redhat.com; receiver=edk2-devel@lists.01.org Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 54EAE211A43A0 for ; Fri, 14 Dec 2018 01:38:26 -0800 (PST) Received: from smtp.corp.redhat.com (int-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.12]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id CF4EF300208D; Fri, 14 Dec 2018 09:38:25 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-120-97.rdu2.redhat.com [10.10.120.97]) by smtp.corp.redhat.com (Postfix) with ESMTP id 3F8B865F6B; Fri, 14 Dec 2018 09:38:24 +0000 (UTC) To: "Ye, Ting" , "Long, Qin" , "Gao, Liming" Cc: "edk2-devel@lists.01.org" References: <20181213074106.10300-1-ting.ye@intel.com> <4A89E2EF3DFEDB4C8BFDE51014F606A14E38C427@SHSMSX104.ccr.corp.intel.com> From: Laszlo Ersek Message-ID: <97de16af-e5db-9363-f637-09217a8bde66@redhat.com> Date: Fri, 14 Dec 2018 10:38:22 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 MIME-Version: 1.0 In-Reply-To: X-Scanned-By: MIMEDefang 2.79 on 10.5.11.12 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.47]); Fri, 14 Dec 2018 09:38:25 +0000 (UTC) Subject: Re: [PATCH] Maintainers.txt: Change package maintainer and reviewer of CryptoPkg. X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 14 Dec 2018 09:38:26 -0000 Content-Type: text/plain; charset=windows-1252 Content-Language: en-US Content-Transfer-Encoding: 7bit On 12/14/18 01:47, Ye, Ting wrote: > Thanks all for the info and follow up. Laszlo, please let me know if there is additional step you think I need follow, thanks. In the future, please let's make sure that the maintainer that is about to give up the role posts the maintenance change *in advance*. It's not a rule I invented or even campaigned for, but it's there, and so we should attempt to follow it. No more comments on this particular update. Thanks, Laszlo > From: Long, Qin > Sent: Friday, December 14, 2018 2:30 AM > To: Gao, Liming ; Laszlo Ersek ; Ye, Ting > Cc: edk2-devel@lists.01.org > Subject: RE: [edk2] [PATCH] Maintainers.txt: Change package maintainer and reviewer of CryptoPkg. > > Confirmed by Long, Qin > > > (And sorry for this rule breaking caused by me. I didn't notice this updates.) > > Best Regards & Thanks, > LONG, Qin > > From: Gao, Liming > Sent: Thursday, December 13, 2018 9:15 PM > To: Laszlo Ersek >; Ye, Ting >; Long, Qin > > Cc: edk2-devel@lists.01.org > Subject: RE: [edk2] [PATCH] Maintainers.txt: Change package maintainer and reviewer of CryptoPkg. > > Laszlo: > Yes. Long, Qin should send this patch. Because Long, Qin changes to another work group for a while, he doesn't work on edk2 project. Ting directly sends the patch to remove his name. I just include Long, Qin, and let him confirm this change. > > Thanks > Liming >> -----Original Message----- >> From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Laszlo Ersek >> Sent: Thursday, December 13, 2018 6:38 PM >> To: Ye, Ting > >> Cc: edk2-devel@lists.01.org >> Subject: Re: [edk2] [PATCH] Maintainers.txt: Change package maintainer and reviewer of CryptoPkg. >> >> Hi Ting, >> >> On 12/13/18 08:41, tye1 wrote: >>> Cc: Gang Wei > >>> Cc: Jian Wang > >>> >>> Contributed-under: TianoCore Contribution Agreement 1.1 >>> Signed-off-by: Ting Ye > >>> --- >>> Maintainers.txt | 3 ++- >>> 1 file changed, 2 insertions(+), 1 deletion(-) >>> >>> diff --git a/Maintainers.txt b/Maintainers.txt >>> index 001d8ba010..d5cb305da9 100644 >>> --- a/Maintainers.txt >>> +++ b/Maintainers.txt >>> @@ -102,8 +102,9 @@ S: Maintained >>> >>> CryptoPkg >>> W: https://github.com/tianocore/tianocore.github.io/wiki/CryptoPkg >>> -M: Qin Long > >>> M: Ting Ye > >>> +R: Gang Wei > >>> +R: Jian Wang > >>> >>> DynamicTablesPkg >>> W: https://github.com/tianocore/tianocore.github.io/wiki/DynamicTablesPkg >>> >> >> This patch does not conform to the rule that we added lately; please see >> commit 9ebef6c0a7d3 ("Maintainers.txt: Add the rule to hand over the >> package maintain role", 2018-11-29). >> >> In other words, the patch should be sent out by Qin Long. Even though >> you co-maintain CryptoPkg with Qin Long, you shouldn't be able to >> deprive Qin Long from the role. >> >> Thanks, >> Laszlo >> _______________________________________________ >> edk2-devel mailing list >> edk2-devel@lists.01.org >> https://lists.01.org/mailman/listinfo/edk2-devel >