From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=fail reason="no key for verify" header.i=@outlook.com header.s=selector1 header.b=m4OtajTO; spf=pass (domain: outlook.com, ip: 40.92.67.70, mailfrom: minnowware@outlook.com) Received: from EUR02-AM5-obe.outbound.protection.outlook.com (EUR02-AM5-obe.outbound.protection.outlook.com [40.92.67.70]) by groups.io with SMTP; Thu, 09 May 2019 06:25:18 -0700 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=outlook.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=Fn770ivbJob/EgYdFMHcBP7VMVOS6C6o5UWbAnC1SXo=; b=m4OtajTOtHqerCFrGdxHqHHIKqJN+gtesg2bz+VKa+qapAQ6cpcp7J8kXViSPKZIbNa5HYvDl+PZ7uqOejwSfYcwDWlgUBHnb9uh42h9cMFkR3D8a8lTrGBu+sxyJ1tPkMlQn95MwomNNOEfjXX+KUeTbyMCBiS5WBWv767FFWKay8jd9BMIfqbty2krLQT/XWxtXvrAn0Ap/uo7+Hm2qLky8iOuLUUMxv3cdqfhaLmY2aOr/3W1t0d0VwkRARynM7JkheJRRN6z/FmWWmGiO6OCOd9hEey5sox/E6U+hVOV0iGDO4XToVsl9Oge5LkW9tZogL0OwQBz0V1SGsKqdg== Received: from AM5EUR02FT065.eop-EUR02.prod.protection.outlook.com (10.152.8.56) by AM5EUR02HT050.eop-EUR02.prod.protection.outlook.com (10.152.9.143) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384) id 15.20.1856.11; Thu, 9 May 2019 13:25:15 +0000 Received: from AM5PR04MB2993.eurprd04.prod.outlook.com (10.152.8.55) by AM5EUR02FT065.mail.protection.outlook.com (10.152.9.25) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1856.11 via Frontend Transport; Thu, 9 May 2019 13:25:15 +0000 Received: from AM5PR04MB2993.eurprd04.prod.outlook.com ([fe80::e15b:160c:f9fd:c286]) by AM5PR04MB2993.eurprd04.prod.outlook.com ([fe80::e15b:160c:f9fd:c286%4]) with mapi id 15.20.1856.016; Thu, 9 May 2019 13:25:15 +0000 From: "Minnow Ware" To: "devel@edk2.groups.io" , "michael.d.kinney@intel.com" , "Carsey, Jaben" , "Bi, Dandan" , Laszlo Ersek , Ard Biesheuvel CC: "Ni, Ray" , Leif Lindholm , "Gao, Liming" , "Richardson, Brian" Subject: Re: [edk2-devel] [patch] ShellBinPkg: Remove ShellBinPkg Thread-Topic: [edk2-devel] [patch] ShellBinPkg: Remove ShellBinPkg Thread-Index: AQHU9OiQ239gHckHt0+/VjjL4v4tKqY/8QiAgAA8kICAAO/ygIAH6gWAgAP4FLqAFJ3WgIAAAiZz Date: Thu, 9 May 2019 13:25:15 +0000 Message-ID: References: <20190417064011.7000-1-dandan.bi@intel.com> <72474caf-c936-1ff7-a92c-b0f279d5418e@redhat.com> <3C0D5C461C9E904E8F62152F6274C0BB40BBD0CA@SHSMSX104.ccr.corp.intel.com>, , In-Reply-To: Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-incomingtopheadermarker: OriginalChecksum:0431365EA96F44B74E574777AD43FDBF3D3A2969B0AC4529168A9674FD9CE511;UpperCasedChecksum:FC3C4EE2137E9D8B059F0C70D7F9AD225AC961F9006C9A693FCE7B6382397A76;SizeAsReceived:7558;Count:43 x-tmn: [0U1GbZU7EclzmX2Sh5gwwwDkBB00jH9b] x-ms-publictraffictype: Email x-incomingheadercount: 43 x-eopattributedmessage: 0 x-microsoft-antispam: BCL:0;PCL:0;RULEID:(2390118)(5050001)(7020095)(20181119110)(201702061078)(5061506573)(5061507331)(1603103135)(2017031320274)(2017031323274)(2017031324274)(2017031322404)(1601125500)(1603101475)(1701031045);SRVR:AM5EUR02HT050; x-ms-traffictypediagnostic: AM5EUR02HT050: x-ms-exchange-purlcount: 10 x-microsoft-antispam-message-info: uUqlOMV9xspnFiudPNjIkFqsJinagZl9dgKh3R1N9V9+SYwaf6FGUdJzcC9wmQ7DLeMn7ONefsK0mXvbtGuipOWKiG3u76ZVLu7/4SEoOZ0uar6D7zp/W2fswCQ0nsPtPU42l6UPavaKQCKRLuQMmrpE+5MNmoV5N4AGDFNty5Se1NsW9PCilK6gDITQ3Fjo MIME-Version: 1.0 X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg: 00000000-0000-0000-0000-000000000000 X-MS-Exchange-CrossTenant-Network-Message-Id: e38a8765-1677-4127-fc2e-08d6d481c5dd X-MS-Exchange-CrossTenant-rms-persistedconsumerorg: 00000000-0000-0000-0000-000000000000 X-MS-Exchange-CrossTenant-originalarrivaltime: 09 May 2019 13:25:15.3973 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Internet X-MS-Exchange-CrossTenant-id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM5EUR02HT050 Content-Language: en-US Content-Type: multipart/alternative; boundary="_000_AM5PR04MB2993E0047FB7D03394BB4809C1320AM5PR04MB2993eurp_" --_000_AM5PR04MB2993E0047FB7D03394BB4809C1320AM5PR04MB2993eurp_ Content-Type: text/plain; charset="Windows-1252" Content-Transfer-Encoding: quoted-printable Hi Mike, thank you for that information. >>From my point of view there should always be public access to a trusted sh= ell 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 variou= s companies, that have never seen an ITP, closed chassis adapter, Arium or Lauterbach d= ebug tool. So there need alternate tools, running on UEFI Shell=85 Thanks, Kilian ________________________________ From: devel@edk2.groups.io on behalf of Michael D K= inney Sent: Wednesday, May 8, 2019 7:29:53 PM To: devel@edk2.groups.io; MinnowWare@outlook.com; Carsey, Jaben; Bi, Danda= n; 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 M= arch that includes the UEFI Shell binary. https://github.com/tianocore/edk2/tree/edk2-stable201903/ShellBinPkg/UefiS= hell/X64 There have been discussions that future edk2-stable tags to provide pre-bu= ilt binaries like the UEFI Shell in the GitHub release page for the edk2-st= able tag. Mike From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of Minn= ow Ware Sent: Tuesday, May 7, 2019 11:35 PM To: devel@edk2.groups.io; Carsey, Jaben ; Bi, Dand= an ; Laszlo Ersek ; Ard Biesheuvel = Cc: edk2-devel-groups-io ; Ni, Ray ; Leif Lindholm ; Gao, Liming ; Richardson, Brian Subject: Re: [edk2-devel] [patch] ShellBinPkg: Remove ShellBinPkg Hi all, does that mean, that SHELL.EFI from a trusted source is currently unavaila= ble for the industry? There are a couple of web sites that refer to =93https://github.com/tianoc= ore/edk2/raw/master/ShellBinPkg/UefiShell/X64=94 (https://www.google.com/search?ei=3DSUvPXPneNIeFmwXQvo6gAg&q=3Dhttps%3A%2F= %2Fgithub.com%2Ftianocore%2Fedk2%2Fraw%2Fmaster%2FShellBinPkg%2FUefiShell%2= FX64&oq=3Dhttps%3A%2F%2Fgithub.com%2Ftianocore%2Fedk2%2Fraw%2Fmaster%2FShel= lBinPkg%2FUefiShell%2FX64&gs_l=3Dpsy-ab.3...3050.3050..3392...0.0..0.102.10= 2.0j1......0....2j1..gws-wiz.9e-I_-IL7Zs) Where get I SHELL.EFI right now? Best regards, Kilian ________________________________ From: devel@edk2.groups.io > on behalf of Carsey, Jaben > 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 > > -----Original Message----- > From: Bi, Dandan > Sent: Wednesday, April 17, 2019 6:12 PM > To: Laszlo Ersek >; Ard Bies= heuvel > > > Cc: edk2-devel-groups-io >; Carsey, Jaben > >; Ni, Ray >; Leif Lindholm > >; Gao, Liming= > > 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@redhat.com] > > Sent: Wednesday, April 17, 2019 6:53 PM > > To: Ard Biesheuvel >; Bi, Dandan > > > > > Cc: edk2-devel-groups-io >; Carsey, Jaben > > >; Ni, Ray >; Leif Lindholm > > >; Gao, Limi= ng > > > 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 > wrote: > > >> > > >> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3D1675 > > >> > > >> 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 > > > >> Cc: Ray Ni > > > >> Cc: Leif Lindholm > > > >> Cc: Ard Biesheuvel > > > >> Cc: Liming Gao > > > >> Cc: Laszlo Ersek > > > >> Signed-off-by: Dandan Bi > > > > > > > Acked-by: Ard Biesheuvel > > > > > > > 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 > > > > > Thanks > > Laszlo --_000_AM5PR04MB2993E0047FB7D03394BB4809C1320AM5PR04MB2993eurp_ Content-Type: text/html; charset="Windows-1252" Content-Transfer-Encoding: quoted-printable

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 boo= ted 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.<= /p>

 

There are at least 1000 BIOS developers in middle a= nd 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 Shel= l=85

 

Thanks,

Kilian

 

From: devel@edk2.groups.io= <devel@edk2.groups.io> on behalf of Michael D Kinney <michael.d.k= inney@intel.com>
Sent: Wednesday, May 8, 2019 7:29:53 PM
To: devel@edk2.groups.io; MinnowWare@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-stabl= e tag from March that includes the UEFI Shell binary.

 

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

 

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

 

Mike

 

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

 

Hi all,

 

does that mean, that SHELL.EFI from a trusted sour= ce is currently unavailable

for the industry?

 

There are a couple of web sites that refer to =93<= a href=3D"https://github.com/tianocore/edk2/raw/master/ShellBinPkg/UefiShel= l/X64">https://github.com/tianocore/edk2/raw/master/ShellBinPkg/UefiShell/X= 64=94

 

(https://www.google.com/search?ei=3DSUvPXPneNIeFmwXQvo6gAg&= ;q=3Dhttps%3A%2F%2Fgithub.com%2Ftianocore%2Fedk2%2Fraw%2Fmaster%2FShellBinP= kg%2FUefiShell%2FX64&oq=3Dhttps%3A%2F%2Fgithub.com%2Ftianocore%2Fedk2%2= Fraw%2Fmaster%2FShellBinPkg%2FUefiShell%2FX64&gs_l=3Dpsy-ab.3...3050.30= 50..3392...0.0..0.102.102.0j1......0....2j1..gws-wiz.9e-I_-IL7Zs)<= /o:p>

 

Where get I SHELL.EFI right now?

 

Best regards,

Kilian

 

 

 

 

From: devel@edk2.groups.io <devel@edk2.groups.io> on behalf o= f Carsey, Jaben <jaben.carsey@= 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 L= aszlo's comment is addressed.
Reviewed-by: Jaben Carsey <ja= ben.carsey@intel.com>

> -----Original Message-----
> From: Bi, Dandan
> Sent: Wednesday, April 17, 2019 6:12 PM
> To: Laszlo Ersek <lersek@redh= at.com>; Ard Biesheuvel
> <ard.biesheuvel@linar= o.org>
> Cc: edk2-devel-groups-io <= devel@edk2.groups.io>; Carsey, Jaben
> <jaben.carsey@intel.com<= /a>>; Ni, Ray <ray.ni@intel.com>; Leif Lindholm
> <leif.lindholm@linaro.= org>; Gao, Liming <liming= .gao@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@redhat.com]
> > Sent: Wednesday, April 17, 2019 6:53 PM
> > To: Ard Biesheuvel <ard.biesheuvel@linaro.org>; Bi, Dandan
> > <dandan.bi@intel.com>
> > Cc: edk2-devel-groups-io <devel@edk2.groups.io>; Carsey, Jaben
> > <jaben.carsey@intel= .com>; Ni, Ray <ray.ni@intel.= com>; Leif Lindholm
> > <leif.lindholm@li= naro.org>; Gao, Liming <l= iming.gao@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@intel.com> wrote:
> > >>
> > >> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3D1675 > > >>
> > >> Remove ShellBinPkg from edk2.
> > >> If developers work on current edk2 master and still wan= t to use
> > >> Shell binary after this removal, they can get it from s= ource code
> > >> in the Assets part of edk2-stable201903 tag release pag= e.
> > >> 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@intel.com>
> > >> Cc: Ray Ni <ray.= ni@intel.com>
> > >> Cc: Leif Lindholm <leif.lindholm@linaro.org>
> > >> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> > >> Cc: Liming Gao <liming.gao@intel.com>
> > >> Cc: Laszlo Ersek <lersek@redhat.com>
> > >> Signed-off-by: Dandan Bi <dandan.bi@intel.com>
> > >
> > > Acked-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> > >
> > > BTW you can use the --no-binary option to git to omit the b= inary
> > > 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 li= ne
> >
> > +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@redhat.com>
> >
> > Thanks
> > Laszlo


--_000_AM5PR04MB2993E0047FB7D03394BB4809C1320AM5PR04MB2993eurp_--