From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: redhat.com, ip: 209.132.183.28, mailfrom: lersek@redhat.com) Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by groups.io with SMTP; Tue, 24 Sep 2019 10:26:48 -0700 Received: from smtp.corp.redhat.com (int-mx06.intmail.prod.int.phx2.redhat.com [10.5.11.16]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 95CDB3090FC6; Tue, 24 Sep 2019 17:26:47 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-120-118.rdu2.redhat.com [10.10.120.118]) by smtp.corp.redhat.com (Postfix) with ESMTP id 5CBDE5C219; Tue, 24 Sep 2019 17:26:46 +0000 (UTC) Subject: Re: [Patch] Maintainers.txt: Move ShellBin maintainers to EDK II Releases section To: Liming Gao , devel@edk2.groups.io Cc: Andrew Fish , Leif Lindholm , Michael D Kinney References: <1569287994-7660-1-git-send-email-liming.gao@intel.com> From: "Laszlo Ersek" Message-ID: Date: Tue, 24 Sep 2019 19:26:45 +0200 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: <1569287994-7660-1-git-send-email-liming.gao@intel.com> X-Scanned-By: MIMEDefang 2.79 on 10.5.11.16 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.43]); Tue, 24 Sep 2019 17:26:47 +0000 (UTC) Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Hi Liming, On 09/24/19 03:19, Liming Gao wrote: > ShellBinPkg is generated for each edk2 stable tag release. > > Cc: Andrew Fish > Cc: Laszlo Ersek > Cc: Leif Lindholm > Cc: Michael D Kinney > Signed-off-by: Liming Gao > --- > 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 > > +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 (Ia32/X64) > +M: Zhichao Gao (Ia32/X64) > +M: Leif Lindholm (ARM/AArch64) > +M: Ard Biesheuvel (ARM/AArch64) > + > EDK II Architectures: > --------------------- > ARM, AARCH64 > @@ -421,13 +427,6 @@ W: https://github.com/tianocore/tianocore.github.io/wiki/ShellPkg > M: Ray Ni > M: Zhichao Gao > > -Maintainers for stable Shell binaries generation > -when need to publish Shell binaries with edk2 release: > -M: Ray Ni (Ia32/X64) > -M: Zhichao Gao (Ia32/X64) > -M: Leif Lindholm (ARM/AArch64) > -M: Ard Biesheuvel (ARM/AArch64) > - > SignedCapsulePkg > F: SignedCapsulePkg/ > W: https://github.com/tianocore/tianocore.github.io/wiki/SignedCapsulePkg >