Re: [PATCH] Maintainers.txt: Change package maintainer and reviewer of CryptoPkg.

["Ye, Ting" <ting.ye@intel.com>]

Thanks all for the info and follow up. Laszlo, please let me know if there is additional step you think I need follow, thanks.

Best Regards,
Ting

From: Long, Qin
Sent: Friday, December 14, 2018 2:30 AM
To: Gao, Liming <liming.gao@intel.com>; Laszlo Ersek <lersek@redhat.com>; Ye, Ting <ting.ye@intel.com>
Cc: edk2-devel@lists.01.org
Subject: RE: [edk2] [PATCH] Maintainers.txt: Change package maintainer and reviewer of CryptoPkg.

Confirmed by Long, Qin <qin.long@intel.com<mailto:qin.long@intel.com>>

(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 <lersek@redhat.com<mailto:lersek@redhat.com>>; Ye, Ting <ting.ye@intel.com<mailto:ting.ye@intel.com>>; Long, Qin <qin.long@intel.com<mailto:qin.long@intel.com>>
Cc: edk2-devel@lists.01.org<mailto: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 <ting.ye@intel.com<mailto:ting.ye@intel.com>>
> Cc: edk2-devel@lists.01.org<mailto: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 <gang.wei@intel.com<mailto:gang.wei@intel.com>>
> > Cc: Jian Wang <jian.j.wang@intel.com<mailto:jian.j.wang@intel.com>>
> >
> > Contributed-under: TianoCore Contribution Agreement 1.1
> > Signed-off-by: Ting Ye <ting.ye@intel.com<mailto:ting.ye@intel.com>>
> > ---
> >  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 <qin.long@intel.com<mailto:qin.long@intel.com>>
> >  M: Ting Ye <ting.ye@intel.com<mailto:ting.ye@intel.com>>
> > +R: Gang Wei <gang.wei@intel.com<mailto:gang.wei@intel.com>>
> > +R: Jian Wang <jian.j.wang@intel.com<mailto:jian.j.wang@intel.com>>
> >
> >  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<mailto:edk2-devel@lists.01.org>
> https://lists.01.org/mailman/listinfo/edk2-devel