* [PATCH] Edk2: Add Zhichao as the maintainer for shellbin relase @ 2019-09-19 5:45 Gao, Zhichao 2019-09-19 5:58 ` Liming Gao 0 siblings, 1 reply; 7+ messages in thread From: Gao, Zhichao @ 2019-09-19 5:45 UTC (permalink / raw) To: devel Cc: Andrew Fish, Laszlo Ersek, Leif Lindholm, Michael D Kinney, Ray Ni, Ard Biesheuvel, Liming Gao Zhichao is responsible for the IA32 and X64 ARCH shell binary generation. Cc: Andrew Fish <afish@apple.com> Cc: Laszlo Ersek <lersek@redhat.com> Cc: Leif Lindholm <leif.lindholm@linaro.org> Cc: Michael D Kinney <michael.d.kinney@intel.com> Cc: Ray Ni <ray.ni@intel.com> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org> Cc: Liming Gao <liming.gao@intel.com> Signed-off-by: Zhichao Gao <zhichao.gao@intel.com> --- Maintainers.txt | 1 + 1 file changed, 1 insertion(+) diff --git a/Maintainers.txt b/Maintainers.txt index dcf81c737a..a0121123ec 100644 --- a/Maintainers.txt +++ b/Maintainers.txt @@ -426,6 +426,7 @@ Maintainers for stable Shell binaries generation when need to publish Shell binaries with edk2 release: M: Jaben Carsey <jaben.carsey@intel.com> (Ia32/X64) M: Ray Ni <ray.ni@intel.com> (Ia32/X64) +M: Zhichao Gao <zhichao.gao@intel.com> (Ia32/X64) M: Leif Lindholm <leif.lindholm@linaro.org> (ARM/AArch64) M: Ard Biesheuvel <ard.biesheuvel@linaro.org> (ARM/AArch64) -- 2.21.0.windows.1 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] Edk2: Add Zhichao as the maintainer for shellbin relase 2019-09-19 5:45 [PATCH] Edk2: Add Zhichao as the maintainer for shellbin relase Gao, Zhichao @ 2019-09-19 5:58 ` Liming Gao 2019-09-19 9:17 ` Leif Lindholm 0 siblings, 1 reply; 7+ messages in thread From: Liming Gao @ 2019-09-19 5:58 UTC (permalink / raw) To: Gao, Zhichao, devel@edk2.groups.io Cc: Andrew Fish, Laszlo Ersek, Leif Lindholm, Kinney, Michael D, Ni, Ray, Ard Biesheuvel Reviewed-by: Liming Gao <liming.gao@intel.com> >-----Original Message----- >From: Gao, Zhichao >Sent: Thursday, September 19, 2019 1:46 PM >To: devel@edk2.groups.io >Cc: Andrew Fish <afish@apple.com>; Laszlo Ersek <lersek@redhat.com>; Leif >Lindholm <leif.lindholm@linaro.org>; Kinney, Michael D ><michael.d.kinney@intel.com>; Ni, Ray <ray.ni@intel.com>; Ard Biesheuvel ><ard.biesheuvel@linaro.org>; Gao, Liming <liming.gao@intel.com> >Subject: [PATCH] Edk2: Add Zhichao as the maintainer for shellbin relase > >Zhichao is responsible for the IA32 and X64 ARCH shell binary generation. > >Cc: Andrew Fish <afish@apple.com> >Cc: Laszlo Ersek <lersek@redhat.com> >Cc: Leif Lindholm <leif.lindholm@linaro.org> >Cc: Michael D Kinney <michael.d.kinney@intel.com> >Cc: Ray Ni <ray.ni@intel.com> >Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org> >Cc: Liming Gao <liming.gao@intel.com> >Signed-off-by: Zhichao Gao <zhichao.gao@intel.com> >--- > Maintainers.txt | 1 + > 1 file changed, 1 insertion(+) > >diff --git a/Maintainers.txt b/Maintainers.txt >index dcf81c737a..a0121123ec 100644 >--- a/Maintainers.txt >+++ b/Maintainers.txt >@@ -426,6 +426,7 @@ Maintainers for stable Shell binaries generation > when need to publish Shell binaries with edk2 release: > M: Jaben Carsey <jaben.carsey@intel.com> (Ia32/X64) > M: Ray Ni <ray.ni@intel.com> (Ia32/X64) >+M: Zhichao Gao <zhichao.gao@intel.com> (Ia32/X64) > M: Leif Lindholm <leif.lindholm@linaro.org> (ARM/AArch64) > M: Ard Biesheuvel <ard.biesheuvel@linaro.org> (ARM/AArch64) > >-- >2.21.0.windows.1 ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] Edk2: Add Zhichao as the maintainer for shellbin relase 2019-09-19 5:58 ` Liming Gao @ 2019-09-19 9:17 ` Leif Lindholm 2019-09-19 14:16 ` Liming Gao 0 siblings, 1 reply; 7+ messages in thread From: Leif Lindholm @ 2019-09-19 9:17 UTC (permalink / raw) To: Gao, Liming Cc: Gao, Zhichao, devel@edk2.groups.io, Andrew Fish, Laszlo Ersek, Kinney, Michael D, Ni, Ray, Ard Biesheuvel I have no objection to this addition, but I do have two questions - and really I realise this is a follow-on for 81a8a52a6bb21 ("ShellBinPkg: Remove ShellBinPkg"): 1) Since ShellBinPkg no longer exists, what is the intent of listing this in the Maintainers.txt file? Generally, this is so that people know who to Cc: on patches - but there will not be any patches here. 2) If Zhichao is now responsible for this, should Jaben and Ray still be listed? Best Regards, Leif On Thu, Sep 19, 2019 at 05:58:16AM +0000, Gao, Liming wrote: > Reviewed-by: Liming Gao <liming.gao@intel.com> > > >-----Original Message----- > >From: Gao, Zhichao > >Sent: Thursday, September 19, 2019 1:46 PM > >To: devel@edk2.groups.io > >Cc: Andrew Fish <afish@apple.com>; Laszlo Ersek <lersek@redhat.com>; Leif > >Lindholm <leif.lindholm@linaro.org>; Kinney, Michael D > ><michael.d.kinney@intel.com>; Ni, Ray <ray.ni@intel.com>; Ard Biesheuvel > ><ard.biesheuvel@linaro.org>; Gao, Liming <liming.gao@intel.com> > >Subject: [PATCH] Edk2: Add Zhichao as the maintainer for shellbin relase > > > >Zhichao is responsible for the IA32 and X64 ARCH shell binary generation. > > > >Cc: Andrew Fish <afish@apple.com> > >Cc: Laszlo Ersek <lersek@redhat.com> > >Cc: Leif Lindholm <leif.lindholm@linaro.org> > >Cc: Michael D Kinney <michael.d.kinney@intel.com> > >Cc: Ray Ni <ray.ni@intel.com> > >Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org> > >Cc: Liming Gao <liming.gao@intel.com> > >Signed-off-by: Zhichao Gao <zhichao.gao@intel.com> > >--- > > Maintainers.txt | 1 + > > 1 file changed, 1 insertion(+) > > > >diff --git a/Maintainers.txt b/Maintainers.txt > >index dcf81c737a..a0121123ec 100644 > >--- a/Maintainers.txt > >+++ b/Maintainers.txt > >@@ -426,6 +426,7 @@ Maintainers for stable Shell binaries generation > > when need to publish Shell binaries with edk2 release: > > M: Jaben Carsey <jaben.carsey@intel.com> (Ia32/X64) > > M: Ray Ni <ray.ni@intel.com> (Ia32/X64) > >+M: Zhichao Gao <zhichao.gao@intel.com> (Ia32/X64) > > M: Leif Lindholm <leif.lindholm@linaro.org> (ARM/AArch64) > > M: Ard Biesheuvel <ard.biesheuvel@linaro.org> (ARM/AArch64) > > > >-- > >2.21.0.windows.1 > ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] Edk2: Add Zhichao as the maintainer for shellbin relase 2019-09-19 9:17 ` Leif Lindholm @ 2019-09-19 14:16 ` Liming Gao 2019-09-19 14:39 ` Leif Lindholm 0 siblings, 1 reply; 7+ messages in thread From: Liming Gao @ 2019-09-19 14:16 UTC (permalink / raw) To: Leif Lindholm Cc: Gao, Zhichao, devel@edk2.groups.io, Andrew Fish, Laszlo Ersek, Kinney, Michael D, Ni, Ray, Ard Biesheuvel Leif: > -----Original Message----- > From: Leif Lindholm [mailto:leif.lindholm@linaro.org] > Sent: Thursday, September 19, 2019 5:18 PM > To: Gao, Liming <liming.gao@intel.com> > Cc: Gao, Zhichao <zhichao.gao@intel.com>; devel@edk2.groups.io; Andrew Fish <afish@apple.com>; Laszlo Ersek <lersek@redhat.com>; > Kinney, Michael D <michael.d.kinney@intel.com>; Ni, Ray <ray.ni@intel.com>; Ard Biesheuvel <ard.biesheuvel@linaro.org> > Subject: Re: [PATCH] Edk2: Add Zhichao as the maintainer for shellbin relase > > I have no objection to this addition, but I do have two questions - and > really I realise this is a follow-on for 81a8a52a6bb21 ("ShellBinPkg: > Remove ShellBinPkg"): > > 1) Since ShellBinPkg no longer exists, what is the intent of listing > this in the Maintainers.txt file? Generally, this is so that people > know who to Cc: on patches - but there will not be any patches here. Here is the info who can be contacted to generate the binary files for the quarter stable tag. > 2) If Zhichao is now responsible for this, should Jaben and Ray still > be listed? > I think Jaben and Ray can be removed. Thanks Liming > Best Regards, > > Leif > > On Thu, Sep 19, 2019 at 05:58:16AM +0000, Gao, Liming wrote: > > Reviewed-by: Liming Gao <liming.gao@intel.com> > > > > >-----Original Message----- > > >From: Gao, Zhichao > > >Sent: Thursday, September 19, 2019 1:46 PM > > >To: devel@edk2.groups.io > > >Cc: Andrew Fish <afish@apple.com>; Laszlo Ersek <lersek@redhat.com>; Leif > > >Lindholm <leif.lindholm@linaro.org>; Kinney, Michael D > > ><michael.d.kinney@intel.com>; Ni, Ray <ray.ni@intel.com>; Ard Biesheuvel > > ><ard.biesheuvel@linaro.org>; Gao, Liming <liming.gao@intel.com> > > >Subject: [PATCH] Edk2: Add Zhichao as the maintainer for shellbin relase > > > > > >Zhichao is responsible for the IA32 and X64 ARCH shell binary generation. > > > > > >Cc: Andrew Fish <afish@apple.com> > > >Cc: Laszlo Ersek <lersek@redhat.com> > > >Cc: Leif Lindholm <leif.lindholm@linaro.org> > > >Cc: Michael D Kinney <michael.d.kinney@intel.com> > > >Cc: Ray Ni <ray.ni@intel.com> > > >Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org> > > >Cc: Liming Gao <liming.gao@intel.com> > > >Signed-off-by: Zhichao Gao <zhichao.gao@intel.com> > > >--- > > > Maintainers.txt | 1 + > > > 1 file changed, 1 insertion(+) > > > > > >diff --git a/Maintainers.txt b/Maintainers.txt > > >index dcf81c737a..a0121123ec 100644 > > >--- a/Maintainers.txt > > >+++ b/Maintainers.txt > > >@@ -426,6 +426,7 @@ Maintainers for stable Shell binaries generation > > > when need to publish Shell binaries with edk2 release: > > > M: Jaben Carsey <jaben.carsey@intel.com> (Ia32/X64) > > > M: Ray Ni <ray.ni@intel.com> (Ia32/X64) > > >+M: Zhichao Gao <zhichao.gao@intel.com> (Ia32/X64) > > > M: Leif Lindholm <leif.lindholm@linaro.org> (ARM/AArch64) > > > M: Ard Biesheuvel <ard.biesheuvel@linaro.org> (ARM/AArch64) > > > > > >-- > > >2.21.0.windows.1 > > ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] Edk2: Add Zhichao as the maintainer for shellbin relase 2019-09-19 14:16 ` Liming Gao @ 2019-09-19 14:39 ` Leif Lindholm 2019-09-19 14:56 ` Liming Gao 0 siblings, 1 reply; 7+ messages in thread From: Leif Lindholm @ 2019-09-19 14:39 UTC (permalink / raw) To: Gao, Liming Cc: Gao, Zhichao, devel@edk2.groups.io, Andrew Fish, Laszlo Ersek, Kinney, Michael D, Ni, Ray, Ard Biesheuvel On Thu, Sep 19, 2019 at 02:16:44PM +0000, Gao, Liming wrote: > Leif: > > > -----Original Message----- > > From: Leif Lindholm [mailto:leif.lindholm@linaro.org] > > Sent: Thursday, September 19, 2019 5:18 PM > > To: Gao, Liming <liming.gao@intel.com> > > Cc: Gao, Zhichao <zhichao.gao@intel.com>; devel@edk2.groups.io; Andrew Fish <afish@apple.com>; Laszlo Ersek <lersek@redhat.com>; > > Kinney, Michael D <michael.d.kinney@intel.com>; Ni, Ray <ray.ni@intel.com>; Ard Biesheuvel <ard.biesheuvel@linaro.org> > > Subject: Re: [PATCH] Edk2: Add Zhichao as the maintainer for shellbin relase > > > > I have no objection to this addition, but I do have two questions - and > > really I realise this is a follow-on for 81a8a52a6bb21 ("ShellBinPkg: > > Remove ShellBinPkg"): > > > > 1) Since ShellBinPkg no longer exists, what is the intent of listing > > this in the Maintainers.txt file? Generally, this is so that people > > know who to Cc: on patches - but there will not be any patches here. > > Here is the info who can be contacted to generate the binary files > for the quarter stable tag. I agree this is important to keep somewhere. But I am unsure if Maintainers.txt is the location for that. Clearly it was when ShellBinPkg existed, but I am not sure now. And yes, this is something I should have realised when ShellBinPkg was removed - but I didn't at that point. So there is no need to hold back this patch. But I think it would make possibly more sense to add this information either into a ShellPkg/Readme.md or the wiki. If we keep the information in Maintainers.txt, I would suggest creating a new section below "EDK II Releases:" with a W: entry pointing somewhere. Best Regards, Leif > > 2) If Zhichao is now responsible for this, should Jaben and Ray still > > be listed? > > > I think Jaben and Ray can be removed. > > Thanks > Liming > > Best Regards, > > > > Leif > > > > On Thu, Sep 19, 2019 at 05:58:16AM +0000, Gao, Liming wrote: > > > Reviewed-by: Liming Gao <liming.gao@intel.com> > > > > > > >-----Original Message----- > > > >From: Gao, Zhichao > > > >Sent: Thursday, September 19, 2019 1:46 PM > > > >To: devel@edk2.groups.io > > > >Cc: Andrew Fish <afish@apple.com>; Laszlo Ersek <lersek@redhat.com>; Leif > > > >Lindholm <leif.lindholm@linaro.org>; Kinney, Michael D > > > ><michael.d.kinney@intel.com>; Ni, Ray <ray.ni@intel.com>; Ard Biesheuvel > > > ><ard.biesheuvel@linaro.org>; Gao, Liming <liming.gao@intel.com> > > > >Subject: [PATCH] Edk2: Add Zhichao as the maintainer for shellbin relase > > > > > > > >Zhichao is responsible for the IA32 and X64 ARCH shell binary generation. > > > > > > > >Cc: Andrew Fish <afish@apple.com> > > > >Cc: Laszlo Ersek <lersek@redhat.com> > > > >Cc: Leif Lindholm <leif.lindholm@linaro.org> > > > >Cc: Michael D Kinney <michael.d.kinney@intel.com> > > > >Cc: Ray Ni <ray.ni@intel.com> > > > >Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org> > > > >Cc: Liming Gao <liming.gao@intel.com> > > > >Signed-off-by: Zhichao Gao <zhichao.gao@intel.com> > > > >--- > > > > Maintainers.txt | 1 + > > > > 1 file changed, 1 insertion(+) > > > > > > > >diff --git a/Maintainers.txt b/Maintainers.txt > > > >index dcf81c737a..a0121123ec 100644 > > > >--- a/Maintainers.txt > > > >+++ b/Maintainers.txt > > > >@@ -426,6 +426,7 @@ Maintainers for stable Shell binaries generation > > > > when need to publish Shell binaries with edk2 release: > > > > M: Jaben Carsey <jaben.carsey@intel.com> (Ia32/X64) > > > > M: Ray Ni <ray.ni@intel.com> (Ia32/X64) > > > >+M: Zhichao Gao <zhichao.gao@intel.com> (Ia32/X64) > > > > M: Leif Lindholm <leif.lindholm@linaro.org> (ARM/AArch64) > > > > M: Ard Biesheuvel <ard.biesheuvel@linaro.org> (ARM/AArch64) > > > > > > > >-- > > > >2.21.0.windows.1 > > > ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] Edk2: Add Zhichao as the maintainer for shellbin relase 2019-09-19 14:39 ` Leif Lindholm @ 2019-09-19 14:56 ` Liming Gao 2019-09-19 19:46 ` Laszlo Ersek 0 siblings, 1 reply; 7+ messages in thread From: Liming Gao @ 2019-09-19 14:56 UTC (permalink / raw) To: Leif Lindholm Cc: Gao, Zhichao, devel@edk2.groups.io, Andrew Fish, Laszlo Ersek, Kinney, Michael D, Ni, Ray, Ard Biesheuvel Leif: > -----Original Message----- > From: Leif Lindholm [mailto:leif.lindholm@linaro.org] > Sent: Thursday, September 19, 2019 10:39 PM > To: Gao, Liming <liming.gao@intel.com> > Cc: Gao, Zhichao <zhichao.gao@intel.com>; devel@edk2.groups.io; Andrew Fish <afish@apple.com>; Laszlo Ersek <lersek@redhat.com>; > Kinney, Michael D <michael.d.kinney@intel.com>; Ni, Ray <ray.ni@intel.com>; Ard Biesheuvel <ard.biesheuvel@linaro.org> > Subject: Re: [PATCH] Edk2: Add Zhichao as the maintainer for shellbin relase > > On Thu, Sep 19, 2019 at 02:16:44PM +0000, Gao, Liming wrote: > > Leif: > > > > > -----Original Message----- > > > From: Leif Lindholm [mailto:leif.lindholm@linaro.org] > > > Sent: Thursday, September 19, 2019 5:18 PM > > > To: Gao, Liming <liming.gao@intel.com> > > > Cc: Gao, Zhichao <zhichao.gao@intel.com>; devel@edk2.groups.io; Andrew Fish <afish@apple.com>; Laszlo Ersek > <lersek@redhat.com>; > > > Kinney, Michael D <michael.d.kinney@intel.com>; Ni, Ray <ray.ni@intel.com>; Ard Biesheuvel <ard.biesheuvel@linaro.org> > > > Subject: Re: [PATCH] Edk2: Add Zhichao as the maintainer for shellbin relase > > > > > > I have no objection to this addition, but I do have two questions - and > > > really I realise this is a follow-on for 81a8a52a6bb21 ("ShellBinPkg: > > > Remove ShellBinPkg"): > > > > > > 1) Since ShellBinPkg no longer exists, what is the intent of listing > > > this in the Maintainers.txt file? Generally, this is so that people > > > know who to Cc: on patches - but there will not be any patches here. > > > > Here is the info who can be contacted to generate the binary files > > for the quarter stable tag. > > I agree this is important to keep somewhere. But I am unsure if > Maintainers.txt is the location for that. Clearly it was when > ShellBinPkg existed, but I am not sure now. > > And yes, this is something I should have realised when ShellBinPkg was > removed - but I didn't at that point. So there is no need to hold > back this patch. > OK. This patch can be pushed first. > But I think it would make possibly more sense to add this information > either into a ShellPkg/Readme.md or the wiki. If we keep the > information in Maintainers.txt, I would suggest creating a new > section below "EDK II Releases:" with a W: entry pointing somewhere. I will take your suggestion to move this info to EDK II Releases: section in Maintainers.txt. I will send another patch for this change. Thanks Liming > > Best Regards, > > Leif > > > > 2) If Zhichao is now responsible for this, should Jaben and Ray still > > > be listed? > > > > > I think Jaben and Ray can be removed. > > > > Thanks > > Liming > > > Best Regards, > > > > > > Leif > > > > > > On Thu, Sep 19, 2019 at 05:58:16AM +0000, Gao, Liming wrote: > > > > Reviewed-by: Liming Gao <liming.gao@intel.com> > > > > > > > > >-----Original Message----- > > > > >From: Gao, Zhichao > > > > >Sent: Thursday, September 19, 2019 1:46 PM > > > > >To: devel@edk2.groups.io > > > > >Cc: Andrew Fish <afish@apple.com>; Laszlo Ersek <lersek@redhat.com>; Leif > > > > >Lindholm <leif.lindholm@linaro.org>; Kinney, Michael D > > > > ><michael.d.kinney@intel.com>; Ni, Ray <ray.ni@intel.com>; Ard Biesheuvel > > > > ><ard.biesheuvel@linaro.org>; Gao, Liming <liming.gao@intel.com> > > > > >Subject: [PATCH] Edk2: Add Zhichao as the maintainer for shellbin relase > > > > > > > > > >Zhichao is responsible for the IA32 and X64 ARCH shell binary generation. > > > > > > > > > >Cc: Andrew Fish <afish@apple.com> > > > > >Cc: Laszlo Ersek <lersek@redhat.com> > > > > >Cc: Leif Lindholm <leif.lindholm@linaro.org> > > > > >Cc: Michael D Kinney <michael.d.kinney@intel.com> > > > > >Cc: Ray Ni <ray.ni@intel.com> > > > > >Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org> > > > > >Cc: Liming Gao <liming.gao@intel.com> > > > > >Signed-off-by: Zhichao Gao <zhichao.gao@intel.com> > > > > >--- > > > > > Maintainers.txt | 1 + > > > > > 1 file changed, 1 insertion(+) > > > > > > > > > >diff --git a/Maintainers.txt b/Maintainers.txt > > > > >index dcf81c737a..a0121123ec 100644 > > > > >--- a/Maintainers.txt > > > > >+++ b/Maintainers.txt > > > > >@@ -426,6 +426,7 @@ Maintainers for stable Shell binaries generation > > > > > when need to publish Shell binaries with edk2 release: > > > > > M: Jaben Carsey <jaben.carsey@intel.com> (Ia32/X64) > > > > > M: Ray Ni <ray.ni@intel.com> (Ia32/X64) > > > > >+M: Zhichao Gao <zhichao.gao@intel.com> (Ia32/X64) > > > > > M: Leif Lindholm <leif.lindholm@linaro.org> (ARM/AArch64) > > > > > M: Ard Biesheuvel <ard.biesheuvel@linaro.org> (ARM/AArch64) > > > > > > > > > >-- > > > > >2.21.0.windows.1 > > > > ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] Edk2: Add Zhichao as the maintainer for shellbin relase 2019-09-19 14:56 ` Liming Gao @ 2019-09-19 19:46 ` Laszlo Ersek 0 siblings, 0 replies; 7+ messages in thread From: Laszlo Ersek @ 2019-09-19 19:46 UTC (permalink / raw) To: Gao, Liming, Leif Lindholm Cc: Gao, Zhichao, devel@edk2.groups.io, Andrew Fish, Kinney, Michael D, Ni, Ray, Ard Biesheuvel On 09/19/19 16:56, Gao, Liming wrote: > Leif: > >> -----Original Message----- >> From: Leif Lindholm [mailto:leif.lindholm@linaro.org] >> Sent: Thursday, September 19, 2019 10:39 PM >> To: Gao, Liming <liming.gao@intel.com> >> Cc: Gao, Zhichao <zhichao.gao@intel.com>; devel@edk2.groups.io; Andrew Fish <afish@apple.com>; Laszlo Ersek <lersek@redhat.com>; >> Kinney, Michael D <michael.d.kinney@intel.com>; Ni, Ray <ray.ni@intel.com>; Ard Biesheuvel <ard.biesheuvel@linaro.org> >> Subject: Re: [PATCH] Edk2: Add Zhichao as the maintainer for shellbin relase >> >> On Thu, Sep 19, 2019 at 02:16:44PM +0000, Gao, Liming wrote: >>> Leif: >>> >>>> -----Original Message----- >>>> From: Leif Lindholm [mailto:leif.lindholm@linaro.org] >>>> Sent: Thursday, September 19, 2019 5:18 PM >>>> To: Gao, Liming <liming.gao@intel.com> >>>> Cc: Gao, Zhichao <zhichao.gao@intel.com>; devel@edk2.groups.io; Andrew Fish <afish@apple.com>; Laszlo Ersek >> <lersek@redhat.com>; >>>> Kinney, Michael D <michael.d.kinney@intel.com>; Ni, Ray <ray.ni@intel.com>; Ard Biesheuvel <ard.biesheuvel@linaro.org> >>>> Subject: Re: [PATCH] Edk2: Add Zhichao as the maintainer for shellbin relase >>>> >>>> I have no objection to this addition, but I do have two questions - and >>>> really I realise this is a follow-on for 81a8a52a6bb21 ("ShellBinPkg: >>>> Remove ShellBinPkg"): >>>> >>>> 1) Since ShellBinPkg no longer exists, what is the intent of listing >>>> this in the Maintainers.txt file? Generally, this is so that people >>>> know who to Cc: on patches - but there will not be any patches here. >>> >>> Here is the info who can be contacted to generate the binary files >>> for the quarter stable tag. >> >> I agree this is important to keep somewhere. But I am unsure if >> Maintainers.txt is the location for that. Clearly it was when >> ShellBinPkg existed, but I am not sure now. >> >> And yes, this is something I should have realised when ShellBinPkg was >> removed - but I didn't at that point. So there is no need to hold >> back this patch. >> > OK. This patch can be pushed first. > >> But I think it would make possibly more sense to add this information >> either into a ShellPkg/Readme.md or the wiki. If we keep the >> information in Maintainers.txt, I would suggest creating a new >> section below "EDK II Releases:" with a W: entry pointing somewhere. > > I will take your suggestion to move this info to EDK II Releases: section in Maintainers.txt. > I will send another patch for this change. Sounds good to me. Thanks Laszlo >>>> 2) If Zhichao is now responsible for this, should Jaben and Ray still >>>> be listed? >>>> >>> I think Jaben and Ray can be removed. >>> >>> Thanks >>> Liming >>>> Best Regards, >>>> >>>> Leif >>>> >>>> On Thu, Sep 19, 2019 at 05:58:16AM +0000, Gao, Liming wrote: >>>>> Reviewed-by: Liming Gao <liming.gao@intel.com> >>>>> >>>>>> -----Original Message----- >>>>>> From: Gao, Zhichao >>>>>> Sent: Thursday, September 19, 2019 1:46 PM >>>>>> To: devel@edk2.groups.io >>>>>> Cc: Andrew Fish <afish@apple.com>; Laszlo Ersek <lersek@redhat.com>; Leif >>>>>> Lindholm <leif.lindholm@linaro.org>; Kinney, Michael D >>>>>> <michael.d.kinney@intel.com>; Ni, Ray <ray.ni@intel.com>; Ard Biesheuvel >>>>>> <ard.biesheuvel@linaro.org>; Gao, Liming <liming.gao@intel.com> >>>>>> Subject: [PATCH] Edk2: Add Zhichao as the maintainer for shellbin relase >>>>>> >>>>>> Zhichao is responsible for the IA32 and X64 ARCH shell binary generation. >>>>>> >>>>>> Cc: Andrew Fish <afish@apple.com> >>>>>> Cc: Laszlo Ersek <lersek@redhat.com> >>>>>> Cc: Leif Lindholm <leif.lindholm@linaro.org> >>>>>> Cc: Michael D Kinney <michael.d.kinney@intel.com> >>>>>> Cc: Ray Ni <ray.ni@intel.com> >>>>>> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org> >>>>>> Cc: Liming Gao <liming.gao@intel.com> >>>>>> Signed-off-by: Zhichao Gao <zhichao.gao@intel.com> >>>>>> --- >>>>>> Maintainers.txt | 1 + >>>>>> 1 file changed, 1 insertion(+) >>>>>> >>>>>> diff --git a/Maintainers.txt b/Maintainers.txt >>>>>> index dcf81c737a..a0121123ec 100644 >>>>>> --- a/Maintainers.txt >>>>>> +++ b/Maintainers.txt >>>>>> @@ -426,6 +426,7 @@ Maintainers for stable Shell binaries generation >>>>>> when need to publish Shell binaries with edk2 release: >>>>>> M: Jaben Carsey <jaben.carsey@intel.com> (Ia32/X64) >>>>>> M: Ray Ni <ray.ni@intel.com> (Ia32/X64) >>>>>> +M: Zhichao Gao <zhichao.gao@intel.com> (Ia32/X64) >>>>>> M: Leif Lindholm <leif.lindholm@linaro.org> (ARM/AArch64) >>>>>> M: Ard Biesheuvel <ard.biesheuvel@linaro.org> (ARM/AArch64) >>>>>> >>>>>> -- >>>>>> 2.21.0.windows.1 >>>>> ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2019-09-19 19:47 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2019-09-19 5:45 [PATCH] Edk2: Add Zhichao as the maintainer for shellbin relase Gao, Zhichao 2019-09-19 5:58 ` Liming Gao 2019-09-19 9:17 ` Leif Lindholm 2019-09-19 14:16 ` Liming Gao 2019-09-19 14:39 ` Leif Lindholm 2019-09-19 14:56 ` Liming Gao 2019-09-19 19:46 ` Laszlo Ersek
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox