* [Patch] Maintainers.txt: Move ShellBin maintainers to EDK II Releases section
@ 2019-09-24 1:19 Liming Gao
2019-09-24 10:41 ` [edk2-devel] " Philippe Mathieu-Daudé
2019-09-24 17:26 ` Laszlo Ersek
0 siblings, 2 replies; 9+ messages in thread
From: Liming Gao @ 2019-09-24 1:19 UTC (permalink / raw)
To: devel; +Cc: Andrew Fish, Laszlo Ersek, Leif Lindholm, Michael D Kinney
ShellBinPkg is generated for each edk2 stable tag release.
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>
Signed-off-by: Liming Gao <liming.gao@intel.com>
---
Maintainers.txt | 13 ++++++-------
1 file changed, 6 insertions(+), 7 deletions(-)
diff --git a/Maintainers.txt b/Maintainers.txt
index f348d70df3..f75b35d5a8 100644
--- a/Maintainers.txt
+++ b/Maintainers.txt
@@ -82,6 +82,12 @@ EDK II Releases:
W: https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Release-Planning
M: Liming Gao <liming.gao@intel.com>
+W: https://github.com/tianocore/edk2/releases/download/edk2-stable20XXXX/ShellBinPkg.zip
+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)
+
EDK II Architectures:
---------------------
ARM, AARCH64
@@ -421,13 +427,6 @@ W: https://github.com/tianocore/tianocore.github.io/wiki/ShellPkg
M: Ray Ni <ray.ni@intel.com>
M: Zhichao Gao <zhichao.gao@intel.com>
-Maintainers for stable Shell binaries generation
-when need to publish Shell binaries with edk2 release:
-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)
-
SignedCapsulePkg
F: SignedCapsulePkg/
W: https://github.com/tianocore/tianocore.github.io/wiki/SignedCapsulePkg
--
2.13.0.windows.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [edk2-devel] [Patch] Maintainers.txt: Move ShellBin maintainers to EDK II Releases section
2019-09-24 1:19 [Patch] Maintainers.txt: Move ShellBin maintainers to EDK II Releases section Liming Gao
@ 2019-09-24 10:41 ` Philippe Mathieu-Daudé
2019-09-24 15:08 ` Liming Gao
2019-09-24 17:17 ` Laszlo Ersek
2019-09-24 17:26 ` Laszlo Ersek
1 sibling, 2 replies; 9+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-09-24 10:41 UTC (permalink / raw)
To: devel, liming.gao, Leif Lindholm
Cc: Andrew Fish, Laszlo Ersek, Michael D Kinney
Hi Liming,
On 9/24/19 3:19 AM, Liming Gao wrote:
> ShellBinPkg is generated for each edk2 stable tag release.
>
> 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>
> Signed-off-by: Liming Gao <liming.gao@intel.com>
> ---
> Maintainers.txt | 13 ++++++-------
> 1 file changed, 6 insertions(+), 7 deletions(-)
>
> diff --git a/Maintainers.txt b/Maintainers.txt
> index f348d70df3..f75b35d5a8 100644
> --- a/Maintainers.txt
> +++ b/Maintainers.txt
> @@ -82,6 +82,12 @@ EDK II Releases:
> W: https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Release-Planning
> M: Liming Gao <liming.gao@intel.com>
>
Shouldn't you keep the F: tag too? Maybe:
ShellPkg
(stable)
F: ShellPkg/
> +W: https://github.com/tianocore/edk2/releases/download/edk2-stable20XXXX/ShellBinPkg.zip
> +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)
> +
> EDK II Architectures:
> ---------------------
> ARM, AARCH64
> @@ -421,13 +427,6 @@ W: https://github.com/tianocore/tianocore.github.io/wiki/ShellPkg
> M: Ray Ni <ray.ni@intel.com>
> M: Zhichao Gao <zhichao.gao@intel.com>
>
> -Maintainers for stable Shell binaries generation
> -when need to publish Shell binaries with edk2 release:
> -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)
> -
> SignedCapsulePkg
> F: SignedCapsulePkg/
> W: https://github.com/tianocore/tianocore.github.io/wiki/SignedCapsulePkg
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [edk2-devel] [Patch] Maintainers.txt: Move ShellBin maintainers to EDK II Releases section
2019-09-24 10:41 ` [edk2-devel] " Philippe Mathieu-Daudé
@ 2019-09-24 15:08 ` Liming Gao
2019-09-24 15:12 ` Philippe Mathieu-Daudé
2019-09-24 17:17 ` Laszlo Ersek
1 sibling, 1 reply; 9+ messages in thread
From: Liming Gao @ 2019-09-24 15:08 UTC (permalink / raw)
To: devel@edk2.groups.io, philmd@redhat.com, Leif Lindholm
Cc: Andrew Fish, Laszlo Ersek, Kinney, Michael D
Philipe:
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Philippe Mathieu-Daudé
> Sent: Tuesday, September 24, 2019 6:41 PM
> To: devel@edk2.groups.io; Gao, Liming <liming.gao@intel.com>; Leif Lindholm <leif.lindholm@linaro.org>
> Cc: Andrew Fish <afish@apple.com>; Laszlo Ersek <lersek@redhat.com>; Kinney, Michael D <michael.d.kinney@intel.com>
> Subject: Re: [edk2-devel] [Patch] Maintainers.txt: Move ShellBin maintainers to EDK II Releases section
>
> Hi Liming,
>
> On 9/24/19 3:19 AM, Liming Gao wrote:
> > ShellBinPkg is generated for each edk2 stable tag release.
> >
> > 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>
> > Signed-off-by: Liming Gao <liming.gao@intel.com>
> > ---
> > Maintainers.txt | 13 ++++++-------
> > 1 file changed, 6 insertions(+), 7 deletions(-)
> >
> > diff --git a/Maintainers.txt b/Maintainers.txt
> > index f348d70df3..f75b35d5a8 100644
> > --- a/Maintainers.txt
> > +++ b/Maintainers.txt
> > @@ -82,6 +82,12 @@ EDK II Releases:
> > W: https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Release-Planning
> > M: Liming Gao <liming.gao@intel.com>
> >
>
> Shouldn't you keep the F: tag too? Maybe:
>
> ShellPkg
> (stable)
> F: ShellPkg/
>
ShellBinPkg doesn't exist in Edk2 repo. It will be generated for stable tag release.
Base on previous discussion, I list it into EDK II Releases section.
Thanks
Liming
>
> > +W: https://github.com/tianocore/edk2/releases/download/edk2-stable20XXXX/ShellBinPkg.zip
> > +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)
> > +
> > EDK II Architectures:
> > ---------------------
> > ARM, AARCH64
> > @@ -421,13 +427,6 @@ W: https://github.com/tianocore/tianocore.github.io/wiki/ShellPkg
> > M: Ray Ni <ray.ni@intel.com>
> > M: Zhichao Gao <zhichao.gao@intel.com>
> >
> > -Maintainers for stable Shell binaries generation
> > -when need to publish Shell binaries with edk2 release:
> > -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)
> > -
> > SignedCapsulePkg
> > F: SignedCapsulePkg/
> > W: https://github.com/tianocore/tianocore.github.io/wiki/SignedCapsulePkg
> >
>
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [edk2-devel] [Patch] Maintainers.txt: Move ShellBin maintainers to EDK II Releases section
2019-09-24 15:08 ` Liming Gao
@ 2019-09-24 15:12 ` Philippe Mathieu-Daudé
2019-09-24 15:17 ` Liming Gao
0 siblings, 1 reply; 9+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-09-24 15:12 UTC (permalink / raw)
To: Gao, Liming, devel@edk2.groups.io, Leif Lindholm
Cc: Andrew Fish, Laszlo Ersek, Kinney, Michael D
On 9/24/19 5:08 PM, Gao, Liming wrote:
> Philipe:
>
>> -----Original Message-----
>> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Philippe Mathieu-Daudé
>> Sent: Tuesday, September 24, 2019 6:41 PM
>> To: devel@edk2.groups.io; Gao, Liming <liming.gao@intel.com>; Leif Lindholm <leif.lindholm@linaro.org>
>> Cc: Andrew Fish <afish@apple.com>; Laszlo Ersek <lersek@redhat.com>; Kinney, Michael D <michael.d.kinney@intel.com>
>> Subject: Re: [edk2-devel] [Patch] Maintainers.txt: Move ShellBin maintainers to EDK II Releases section
>>
>> Hi Liming,
>>
>> On 9/24/19 3:19 AM, Liming Gao wrote:
>>> ShellBinPkg is generated for each edk2 stable tag release.
>>>
>>> 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>
>>> Signed-off-by: Liming Gao <liming.gao@intel.com>
>>> ---
>>> Maintainers.txt | 13 ++++++-------
>>> 1 file changed, 6 insertions(+), 7 deletions(-)
>>>
>>> diff --git a/Maintainers.txt b/Maintainers.txt
>>> index f348d70df3..f75b35d5a8 100644
>>> --- a/Maintainers.txt
>>> +++ b/Maintainers.txt
>>> @@ -82,6 +82,12 @@ EDK II Releases:
>>> W: https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Release-Planning
>>> M: Liming Gao <liming.gao@intel.com>
>>>
>>
>> Shouldn't you keep the F: tag too? Maybe:
>>
>> ShellPkg
>> (stable)
>> F: ShellPkg/
>>
> ShellBinPkg doesn't exist in Edk2 repo. It will be generated for stable tag release.
> Base on previous discussion, I list it into EDK II Releases section.
I should have written "F: ShellBinPkg/", I was thinking someone run this
command in tarball extracted from stable:
./BaseTools/Scripts/GetMaintainer.py ShellBinPkg/...
>
> Thanks
> Liming
>>
>>> +W: https://github.com/tianocore/edk2/releases/download/edk2-stable20XXXX/ShellBinPkg.zip
>>> +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)
>>> +
>>> EDK II Architectures:
>>> ---------------------
>>> ARM, AARCH64
>>> @@ -421,13 +427,6 @@ W: https://github.com/tianocore/tianocore.github.io/wiki/ShellPkg
>>> M: Ray Ni <ray.ni@intel.com>
>>> M: Zhichao Gao <zhichao.gao@intel.com>
>>>
>>> -Maintainers for stable Shell binaries generation
>>> -when need to publish Shell binaries with edk2 release:
>>> -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)
>>> -
>>> SignedCapsulePkg
>>> F: SignedCapsulePkg/
>>> W: https://github.com/tianocore/tianocore.github.io/wiki/SignedCapsulePkg
>>>
>>
>>
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [edk2-devel] [Patch] Maintainers.txt: Move ShellBin maintainers to EDK II Releases section
2019-09-24 15:12 ` Philippe Mathieu-Daudé
@ 2019-09-24 15:17 ` Liming Gao
2019-09-24 15:25 ` Philippe Mathieu-Daudé
0 siblings, 1 reply; 9+ messages in thread
From: Liming Gao @ 2019-09-24 15:17 UTC (permalink / raw)
To: Philippe Mathieu-Daudé, devel@edk2.groups.io, Leif Lindholm
Cc: Andrew Fish, Laszlo Ersek, Kinney, Michael D
> -----Original Message-----
> From: Philippe Mathieu-Daudé <philmd@redhat.com>
> Sent: Tuesday, September 24, 2019 11:13 PM
> To: Gao, Liming <liming.gao@intel.com>; devel@edk2.groups.io; Leif Lindholm <leif.lindholm@linaro.org>
> Cc: Andrew Fish <afish@apple.com>; Laszlo Ersek <lersek@redhat.com>; Kinney, Michael D <michael.d.kinney@intel.com>
> Subject: Re: [edk2-devel] [Patch] Maintainers.txt: Move ShellBin maintainers to EDK II Releases section
>
> On 9/24/19 5:08 PM, Gao, Liming wrote:
> > Philipe:
> >
> >> -----Original Message-----
> >> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Philippe Mathieu-Daudé
> >> Sent: Tuesday, September 24, 2019 6:41 PM
> >> To: devel@edk2.groups.io; Gao, Liming <liming.gao@intel.com>; Leif Lindholm <leif.lindholm@linaro.org>
> >> Cc: Andrew Fish <afish@apple.com>; Laszlo Ersek <lersek@redhat.com>; Kinney, Michael D <michael.d.kinney@intel.com>
> >> Subject: Re: [edk2-devel] [Patch] Maintainers.txt: Move ShellBin maintainers to EDK II Releases section
> >>
> >> Hi Liming,
> >>
> >> On 9/24/19 3:19 AM, Liming Gao wrote:
> >>> ShellBinPkg is generated for each edk2 stable tag release.
> >>>
> >>> 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>
> >>> Signed-off-by: Liming Gao <liming.gao@intel.com>
> >>> ---
> >>> Maintainers.txt | 13 ++++++-------
> >>> 1 file changed, 6 insertions(+), 7 deletions(-)
> >>>
> >>> diff --git a/Maintainers.txt b/Maintainers.txt
> >>> index f348d70df3..f75b35d5a8 100644
> >>> --- a/Maintainers.txt
> >>> +++ b/Maintainers.txt
> >>> @@ -82,6 +82,12 @@ EDK II Releases:
> >>> W: https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Release-Planning
> >>> M: Liming Gao <liming.gao@intel.com>
> >>>
> >>
> >> Shouldn't you keep the F: tag too? Maybe:
> >>
> >> ShellPkg
> >> (stable)
> >> F: ShellPkg/
> >>
> > ShellBinPkg doesn't exist in Edk2 repo. It will be generated for stable tag release.
> > Base on previous discussion, I list it into EDK II Releases section.
>
> I should have written "F: ShellBinPkg/", I was thinking someone run this
> command in tarball extracted from stable:
>
> ./BaseTools/Scripts/GetMaintainer.py ShellBinPkg/...
GetMaintainer.py bases on git commit. But, there is no git commit for ShellBinPkg.
So, the above command doesn't work.
Thanks
Liming
>
> >
> > Thanks
> > Liming
> >>
> >>> +W: https://github.com/tianocore/edk2/releases/download/edk2-stable20XXXX/ShellBinPkg.zip
> >>> +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)
> >>> +
> >>> EDK II Architectures:
> >>> ---------------------
> >>> ARM, AARCH64
> >>> @@ -421,13 +427,6 @@ W: https://github.com/tianocore/tianocore.github.io/wiki/ShellPkg
> >>> M: Ray Ni <ray.ni@intel.com>
> >>> M: Zhichao Gao <zhichao.gao@intel.com>
> >>>
> >>> -Maintainers for stable Shell binaries generation
> >>> -when need to publish Shell binaries with edk2 release:
> >>> -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)
> >>> -
> >>> SignedCapsulePkg
> >>> F: SignedCapsulePkg/
> >>> W: https://github.com/tianocore/tianocore.github.io/wiki/SignedCapsulePkg
> >>>
> >>
> >>
> >
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [edk2-devel] [Patch] Maintainers.txt: Move ShellBin maintainers to EDK II Releases section
2019-09-24 15:17 ` Liming Gao
@ 2019-09-24 15:25 ` Philippe Mathieu-Daudé
0 siblings, 0 replies; 9+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-09-24 15:25 UTC (permalink / raw)
To: Gao, Liming, devel@edk2.groups.io, Leif Lindholm
Cc: Andrew Fish, Laszlo Ersek, Kinney, Michael D
On 9/24/19 5:17 PM, Gao, Liming wrote:
>> -----Original Message-----
>> From: Philippe Mathieu-Daudé <philmd@redhat.com>
>> Sent: Tuesday, September 24, 2019 11:13 PM
>> To: Gao, Liming <liming.gao@intel.com>; devel@edk2.groups.io; Leif Lindholm <leif.lindholm@linaro.org>
>> Cc: Andrew Fish <afish@apple.com>; Laszlo Ersek <lersek@redhat.com>; Kinney, Michael D <michael.d.kinney@intel.com>
>> Subject: Re: [edk2-devel] [Patch] Maintainers.txt: Move ShellBin maintainers to EDK II Releases section
>>
>> On 9/24/19 5:08 PM, Gao, Liming wrote:
>>> Philipe:
>>>
>>>> -----Original Message-----
>>>> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Philippe Mathieu-Daudé
>>>> Sent: Tuesday, September 24, 2019 6:41 PM
>>>> To: devel@edk2.groups.io; Gao, Liming <liming.gao@intel.com>; Leif Lindholm <leif.lindholm@linaro.org>
>>>> Cc: Andrew Fish <afish@apple.com>; Laszlo Ersek <lersek@redhat.com>; Kinney, Michael D <michael.d.kinney@intel.com>
>>>> Subject: Re: [edk2-devel] [Patch] Maintainers.txt: Move ShellBin maintainers to EDK II Releases section
>>>>
>>>> Hi Liming,
>>>>
>>>> On 9/24/19 3:19 AM, Liming Gao wrote:
>>>>> ShellBinPkg is generated for each edk2 stable tag release.
>>>>>
>>>>> 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>
>>>>> Signed-off-by: Liming Gao <liming.gao@intel.com>
>>>>> ---
>>>>> Maintainers.txt | 13 ++++++-------
>>>>> 1 file changed, 6 insertions(+), 7 deletions(-)
>>>>>
>>>>> diff --git a/Maintainers.txt b/Maintainers.txt
>>>>> index f348d70df3..f75b35d5a8 100644
>>>>> --- a/Maintainers.txt
>>>>> +++ b/Maintainers.txt
>>>>> @@ -82,6 +82,12 @@ EDK II Releases:
>>>>> W: https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Release-Planning
>>>>> M: Liming Gao <liming.gao@intel.com>
>>>>>
>>>>
>>>> Shouldn't you keep the F: tag too? Maybe:
>>>>
>>>> ShellPkg
>>>> (stable)
>>>> F: ShellPkg/
>>>>
>>> ShellBinPkg doesn't exist in Edk2 repo. It will be generated for stable tag release.
>>> Base on previous discussion, I list it into EDK II Releases section.
>>
>> I should have written "F: ShellBinPkg/", I was thinking someone run this
>> command in tarball extracted from stable:
>>
>> ./BaseTools/Scripts/GetMaintainer.py ShellBinPkg/...
>
> GetMaintainer.py bases on git commit. But, there is no git commit for ShellBinPkg.
> So, the above command doesn't work.
OK, thanks for clarifying!
>
> Thanks
> Liming
>>
>>>
>>> Thanks
>>> Liming
>>>>
>>>>> +W: https://github.com/tianocore/edk2/releases/download/edk2-stable20XXXX/ShellBinPkg.zip
>>>>> +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)
>>>>> +
>>>>> EDK II Architectures:
>>>>> ---------------------
>>>>> ARM, AARCH64
>>>>> @@ -421,13 +427,6 @@ W: https://github.com/tianocore/tianocore.github.io/wiki/ShellPkg
>>>>> M: Ray Ni <ray.ni@intel.com>
>>>>> M: Zhichao Gao <zhichao.gao@intel.com>
>>>>>
>>>>> -Maintainers for stable Shell binaries generation
>>>>> -when need to publish Shell binaries with edk2 release:
>>>>> -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)
>>>>> -
>>>>> SignedCapsulePkg
>>>>> F: SignedCapsulePkg/
>>>>> W: https://github.com/tianocore/tianocore.github.io/wiki/SignedCapsulePkg
>>>>>
>>>>
>>>>
>>>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [edk2-devel] [Patch] Maintainers.txt: Move ShellBin maintainers to EDK II Releases section
2019-09-24 10:41 ` [edk2-devel] " Philippe Mathieu-Daudé
2019-09-24 15:08 ` Liming Gao
@ 2019-09-24 17:17 ` Laszlo Ersek
1 sibling, 0 replies; 9+ messages in thread
From: Laszlo Ersek @ 2019-09-24 17:17 UTC (permalink / raw)
To: Philippe Mathieu-Daudé, devel, liming.gao, Leif Lindholm
Cc: Andrew Fish, Michael D Kinney
On 09/24/19 12:41, Philippe Mathieu-Daudé wrote:
> Hi Liming,
>
> On 9/24/19 3:19 AM, Liming Gao wrote:
>> ShellBinPkg is generated for each edk2 stable tag release.
>>
>> 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>
>> Signed-off-by: Liming Gao <liming.gao@intel.com>
>> ---
>> Maintainers.txt | 13 ++++++-------
>> 1 file changed, 6 insertions(+), 7 deletions(-)
>>
>> diff --git a/Maintainers.txt b/Maintainers.txt
>> index f348d70df3..f75b35d5a8 100644
>> --- a/Maintainers.txt
>> +++ b/Maintainers.txt
>> @@ -82,6 +82,12 @@ EDK II Releases:
>> W: https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Release-Planning
>> M: Liming Gao <liming.gao@intel.com>
>>
>
> Shouldn't you keep the F: tag too? Maybe:
>
> ShellPkg
> (stable)
> F: ShellPkg/
I think we should keep "F:" only if the related "M:" entries work as
expected, with the "BaseTools/Scripts/GetMaintainer.py" script.
(I haven't checked.)
Thanks
Laszlo
>
>
>> +W: https://github.com/tianocore/edk2/releases/download/edk2-stable20XXXX/ShellBinPkg.zip
>> +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)
>> +
>> EDK II Architectures:
>> ---------------------
>> ARM, AARCH64
>> @@ -421,13 +427,6 @@ W: https://github.com/tianocore/tianocore.github.io/wiki/ShellPkg
>> M: Ray Ni <ray.ni@intel.com>
>> M: Zhichao Gao <zhichao.gao@intel.com>
>>
>> -Maintainers for stable Shell binaries generation
>> -when need to publish Shell binaries with edk2 release:
>> -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)
>> -
>> SignedCapsulePkg
>> F: SignedCapsulePkg/
>> W: https://github.com/tianocore/tianocore.github.io/wiki/SignedCapsulePkg
>>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Patch] Maintainers.txt: Move ShellBin maintainers to EDK II Releases section
2019-09-24 1:19 [Patch] Maintainers.txt: Move ShellBin maintainers to EDK II Releases section Liming Gao
2019-09-24 10:41 ` [edk2-devel] " Philippe Mathieu-Daudé
@ 2019-09-24 17:26 ` Laszlo Ersek
2019-09-25 0:43 ` Liming Gao
1 sibling, 1 reply; 9+ messages in thread
From: Laszlo Ersek @ 2019-09-24 17:26 UTC (permalink / raw)
To: Liming Gao, devel; +Cc: Andrew Fish, Leif Lindholm, Michael D Kinney
Hi Liming,
On 09/24/19 03:19, Liming Gao wrote:
> ShellBinPkg is generated for each edk2 stable tag release.
>
> 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>
> Signed-off-by: Liming Gao <liming.gao@intel.com>
> ---
> Maintainers.txt | 13 ++++++-------
> 1 file changed, 6 insertions(+), 7 deletions(-)
>
> diff --git a/Maintainers.txt b/Maintainers.txt
> index f348d70df3..f75b35d5a8 100644
> --- a/Maintainers.txt
> +++ b/Maintainers.txt
> @@ -82,6 +82,12 @@ EDK II Releases:
> W: https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Release-Planning
> M: Liming Gao <liming.gao@intel.com>
>
> +W: https://github.com/tianocore/edk2/releases/download/edk2-stable20XXXX/ShellBinPkg.zip
I'm not a fan of this link. It contains "XXXX", therefore it cannot work
with a simple click (or copy & paste). And it's not easy to notice the
"XXXX" part, and to substitute a valid stable tag identifier for it.
All the links we provide should work without manual tweaking, in my
opinion. They might not provide an immediately downloadable file (which
is fine with me), but they should not result in a 404.
How about the following:
UEFI Shell Binaries (ShellBinPkg.zip) from EDK II Releases:
-----------------------------------------------------------
W: https://github.com/tianocore/edk2/releases/
M: ...
M: ...
M: ...
M: ...
This allows users to go to the releases page, and search that page for
"ShellBinPkg.zip".
For me, the "Assets" block of the latest release is open immediately,
upon loading the page, and so searching the page for "ShellBinPkg.zip"
provides a hit at once. If a user wants an older build, they can locate
the Assets blocks under older releases, and look for "ShellBinPkg.zip"
there.
Thanks
Laszlo
> +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)
> +
> EDK II Architectures:
> ---------------------
> ARM, AARCH64
> @@ -421,13 +427,6 @@ W: https://github.com/tianocore/tianocore.github.io/wiki/ShellPkg
> M: Ray Ni <ray.ni@intel.com>
> M: Zhichao Gao <zhichao.gao@intel.com>
>
> -Maintainers for stable Shell binaries generation
> -when need to publish Shell binaries with edk2 release:
> -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)
> -
> SignedCapsulePkg
> F: SignedCapsulePkg/
> W: https://github.com/tianocore/tianocore.github.io/wiki/SignedCapsulePkg
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Patch] Maintainers.txt: Move ShellBin maintainers to EDK II Releases section
2019-09-24 17:26 ` Laszlo Ersek
@ 2019-09-25 0:43 ` Liming Gao
0 siblings, 0 replies; 9+ messages in thread
From: Liming Gao @ 2019-09-25 0:43 UTC (permalink / raw)
To: Laszlo Ersek, devel@edk2.groups.io
Cc: Andrew Fish, Leif Lindholm, Kinney, Michael D
Laszlo:
>-----Original Message-----
>From: Laszlo Ersek [mailto:lersek@redhat.com]
>Sent: Wednesday, September 25, 2019 1:27 AM
>To: Gao, Liming <liming.gao@intel.com>; devel@edk2.groups.io
>Cc: Andrew Fish <afish@apple.com>; Leif Lindholm <leif.lindholm@linaro.org>;
>Kinney, Michael D <michael.d.kinney@intel.com>
>Subject: Re: [Patch] Maintainers.txt: Move ShellBin maintainers to EDK II
>Releases section
>
>Hi Liming,
>
>On 09/24/19 03:19, Liming Gao wrote:
>> ShellBinPkg is generated for each edk2 stable tag release.
>>
>> 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>
>> Signed-off-by: Liming Gao <liming.gao@intel.com>
>> ---
>> Maintainers.txt | 13 ++++++-------
>> 1 file changed, 6 insertions(+), 7 deletions(-)
>>
>> diff --git a/Maintainers.txt b/Maintainers.txt
>> index f348d70df3..f75b35d5a8 100644
>> --- a/Maintainers.txt
>> +++ b/Maintainers.txt
>> @@ -82,6 +82,12 @@ EDK II Releases:
>> W: https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Release-
>Planning
>> M: Liming Gao <liming.gao@intel.com>
>>
>> +W: https://github.com/tianocore/edk2/releases/download/edk2-
>stable20XXXX/ShellBinPkg.zip
>
>I'm not a fan of this link. It contains "XXXX", therefore it cannot work
>with a simple click (or copy & paste). And it's not easy to notice the
>"XXXX" part, and to substitute a valid stable tag identifier for it.
>
>All the links we provide should work without manual tweaking, in my
>opinion. They might not provide an immediately downloadable file (which
>is fine with me), but they should not result in a 404.
>
>How about the following:
>
>UEFI Shell Binaries (ShellBinPkg.zip) from EDK II Releases:
>-----------------------------------------------------------
>W: https://github.com/tianocore/edk2/releases/
>M: ...
>M: ...
>M: ...
>M: ...
>
Good suggestion. I will use this style.
Thanks
Liming
>This allows users to go to the releases page, and search that page for
>"ShellBinPkg.zip".
>
>For me, the "Assets" block of the latest release is open immediately,
>upon loading the page, and so searching the page for "ShellBinPkg.zip"
>provides a hit at once. If a user wants an older build, they can locate
>the Assets blocks under older releases, and look for "ShellBinPkg.zip"
>there.
>
>Thanks
>Laszlo
>
>> +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)
>> +
>> EDK II Architectures:
>> ---------------------
>> ARM, AARCH64
>> @@ -421,13 +427,6 @@ W:
>https://github.com/tianocore/tianocore.github.io/wiki/ShellPkg
>> M: Ray Ni <ray.ni@intel.com>
>> M: Zhichao Gao <zhichao.gao@intel.com>
>>
>> -Maintainers for stable Shell binaries generation
>> -when need to publish Shell binaries with edk2 release:
>> -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)
>> -
>> SignedCapsulePkg
>> F: SignedCapsulePkg/
>> W:
>https://github.com/tianocore/tianocore.github.io/wiki/SignedCapsulePkg
>>
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2019-09-25 0:43 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-09-24 1:19 [Patch] Maintainers.txt: Move ShellBin maintainers to EDK II Releases section Liming Gao
2019-09-24 10:41 ` [edk2-devel] " Philippe Mathieu-Daudé
2019-09-24 15:08 ` Liming Gao
2019-09-24 15:12 ` Philippe Mathieu-Daudé
2019-09-24 15:17 ` Liming Gao
2019-09-24 15:25 ` Philippe Mathieu-Daudé
2019-09-24 17:17 ` Laszlo Ersek
2019-09-24 17:26 ` Laszlo Ersek
2019-09-25 0:43 ` Liming Gao
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox