[edk2-devel] [patch] ShellBinPkg: Remove ShellBinPkg

Thu May 9 13:25:15 UTC 2019

Hi Mike,

thank you for that information.

>From my point of view there should always be public access to a trusted shell binary.

The UEFI Shell as a successor of MSDOS, that is booted instantly, has full HW access
shall be usable for production, quality assurance, provisioning in the PC industry or just
to support the BIOS development process itself with tools, before Windows is running on
a particular platform, and RWEverything is usable.

There are at least 1000 BIOS developers in middle and far east from various companies,
that have never seen an ITP, closed chassis adapter, Arium or Lauterbach debug tool.
So there need alternate tools, running on UEFI Shell…

Thanks,
Kilian

________________________________
From: devel at edk2.groups.io <devel at edk2.groups.io> on behalf of Michael D Kinney <michael.d.kinney at intel.com>
Sent: Wednesday, May 8, 2019 7:29:53 PM
To: devel at edk2.groups.io; MinnowWare at outlook.com; Carsey, Jaben; Bi, Dandan; Laszlo Ersek; Ard Biesheuvel; Kinney, Michael D
Cc: Ni, Ray; Leif Lindholm; Gao, Liming; Richardson, Brian
Subject: Re: [edk2-devel] [patch] ShellBinPkg: Remove ShellBinPkg

Killian,

The immediate replacement is to use the link to the edk2-stable tag from March that includes the UEFI Shell binary.

https://github.com/tianocore/edk2/tree/edk2-stable201903/ShellBinPkg/UefiShell/X64

There have been discussions that future edk2-stable tags to provide pre-built binaries like the UEFI Shell in the GitHub release page for the edk2-stable tag.

Mike

From: devel at edk2.groups.io [mailto:devel at edk2.groups.io] On Behalf Of Minnow Ware
Sent: Tuesday, May 7, 2019 11:35 PM
To: devel at edk2.groups.io; Carsey, Jaben <jaben.carsey at intel.com>; Bi, Dandan <dandan.bi at intel.com>; Laszlo Ersek <lersek at redhat.com>; Ard Biesheuvel <ard.biesheuvel at linaro.org>
Cc: edk2-devel-groups-io <devel at edk2.groups.io>; Ni, Ray <ray.ni at intel.com>; Leif Lindholm <leif.lindholm at linaro.org>; Gao, Liming <liming.gao at intel.com>; Richardson, Brian <brian.richardson at intel.com>
Subject: Re: [edk2-devel] [patch] ShellBinPkg: Remove ShellBinPkg

Hi all,

does that mean, that SHELL.EFI from a trusted source is currently unavailable

for the industry?

There are a couple of web sites that refer to “https://github.com/tianocore/edk2/raw/master/ShellBinPkg/UefiShell/X64”

(https://www.google.com/search?ei=SUvPXPneNIeFmwXQvo6gAg&q=https%3A%2F%2Fgithub.com%2Ftianocore%2Fedk2%2Fraw%2Fmaster%2FShellBinPkg%2FUefiShell%2FX64&oq=https%3A%2F%2Fgithub.com%2Ftianocore%2Fedk2%2Fraw%2Fmaster%2FShellBinPkg%2FUefiShell%2FX64&gs_l=psy-ab.3...3050.3050..3392...0.0..0.102.102.0j1......0....2j1..gws-wiz.9e-I_-IL7Zs)

Where get I SHELL.EFI right now?

Best regards,

Kilian

________________________________
From: devel at edk2.groups.io<mailto:devel at edk2.groups.io> <devel at edk2.groups.io<mailto:devel at edk2.groups.io>> on behalf of Carsey, Jaben <jaben.carsey at intel.com<mailto:jaben.carsey at intel.com>>
Sent: Tuesday, April 23, 2019 4:03:05 AM
To: Bi, Dandan; Laszlo Ersek; Ard Biesheuvel
Cc: edk2-devel-groups-io; Ni, Ray; Leif Lindholm; Gao, Liming
Subject: Re: [edk2-devel] [patch] ShellBinPkg: Remove ShellBinPkg

Assuming Laszlo's comment is addressed.
Reviewed-by: Jaben Carsey <jaben.carsey at intel.com<mailto:jaben.carsey at intel.com>>

> -----Original Message-----
> From: Bi, Dandan
> Sent: Wednesday, April 17, 2019 6:12 PM
> To: Laszlo Ersek <lersek at redhat.com<mailto:lersek at redhat.com>>; Ard Biesheuvel
> <ard.biesheuvel at linaro.org<mailto:ard.biesheuvel at linaro.org>>
> Cc: edk2-devel-groups-io <devel at edk2.groups.io<mailto:devel at edk2.groups.io>>; Carsey, Jaben
> <jaben.carsey at intel.com<mailto:jaben.carsey at intel.com>>; Ni, Ray <ray.ni at intel.com<mailto:ray.ni at intel.com>>; Leif Lindholm
> <leif.lindholm at linaro.org<mailto:leif.lindholm at linaro.org>>; Gao, Liming <liming.gao at intel.com<mailto:liming.gao at intel.com>>
> Subject: RE: [patch] ShellBinPkg: Remove ShellBinPkg
> Importance: High
>
> Thank you all.
> I will update the patch according to Laszlo's comments before commit the
> patch.
>
>
> Thanks,
> Dandan
> > -----Original Message-----
> > From: Laszlo Ersek [mailto:lersek at redhat.com]
> > Sent: Wednesday, April 17, 2019 6:53 PM
> > To: Ard Biesheuvel <ard.biesheuvel at linaro.org<mailto:ard.biesheuvel at linaro.org>>; Bi, Dandan
> > <dandan.bi at intel.com<mailto:dandan.bi at intel.com>>
> > Cc: edk2-devel-groups-io <devel at edk2.groups.io<mailto:devel at edk2.groups.io>>; Carsey, Jaben
> > <jaben.carsey at intel.com<mailto:jaben.carsey at intel.com>>; Ni, Ray <ray.ni at intel.com<mailto:ray.ni at intel.com>>; Leif Lindholm
> > <leif.lindholm at linaro.org<mailto:leif.lindholm at linaro.org>>; Gao, Liming <liming.gao at intel.com<mailto:liming.gao at intel.com>>
> > Subject: Re: [patch] ShellBinPkg: Remove ShellBinPkg
> >
> > On 04/17/19 09:16, Ard Biesheuvel wrote:
> > > On Tue, 16 Apr 2019 at 23:40, Dandan Bi <dandan.bi at intel.com<mailto:dandan.bi at intel.com>> wrote:
> > >>
> > >> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1675
> > >>
> > >> Remove ShellBinPkg from edk2.
> > >> If developers work on current edk2 master and still want to use
> > >> Shell binary after this removal, they can get it from source code
> > >> in the Assets part of edk2-stable201903 tag release page.
> > >> And from next release (edk2-stable201905 tag), we will publish all
> > >> Shell binaries to the according Assets part directly.
> > >> edk2 release link: https://github.com/tianocore/edk2/releases
> > >>
> > >> Cc: Jaben Carsey <jaben.carsey at intel.com<mailto:jaben.carsey at intel.com>>
> > >> Cc: Ray Ni <ray.ni at intel.com<mailto:ray.ni at intel.com>>
> > >> Cc: Leif Lindholm <leif.lindholm at linaro.org<mailto:leif.lindholm at linaro.org>>
> > >> Cc: Ard Biesheuvel <ard.biesheuvel at linaro.org<mailto:ard.biesheuvel at linaro.org>>
> > >> Cc: Liming Gao <liming.gao at intel.com<mailto:liming.gao at intel.com>>
> > >> Cc: Laszlo Ersek <lersek at redhat.com<mailto:lersek at redhat.com>>
> > >> Signed-off-by: Dandan Bi <dandan.bi at intel.com<mailto:dandan.bi at intel.com>>
> > >
> > > Acked-by: Ard Biesheuvel <ard.biesheuvel at linaro.org<mailto:ard.biesheuvel at linaro.org>>
> > >
> > > BTW you can use the --no-binary option to git to omit the binary
> > > diffs in the patch. Especially when deleting binaries, that saves a
> > > lot of hassle dealing with a multimegabyte patch email.
> >
> > +1
> >
> > For the patch, I have a request towards Dandan: regarding the line
> >
> > +Maintainers for stable Shell binaries generation when need to publish
> > +Shell
> > binaries with edk2 release:
> >
> > please:
> >
> > (1) wrap it with an 80-char margin,
> > (2) please insert a blank line just above it.
> >
> > With those:
> >
> > Acked-by: Laszlo Ersek <lersek at redhat.com<mailto:lersek at redhat.com>>
> >
> > Thanks
> > Laszlo

-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#40354): https://edk2.groups.io/g/devel/message/40354
Mute This Topic: https://groups.io/mt/31209298/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/edk2-devel-archive/attachments/20190509/63095d65/attachment.htm>