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=cx9JD8ZJ; spf=pass (domain: outlook.com, ip: 40.92.67.17, mailfrom: minnowware@outlook.com) Received: from EUR02-AM5-obe.outbound.protection.outlook.com (EUR02-AM5-obe.outbound.protection.outlook.com [40.92.67.17]) by groups.io with SMTP; Tue, 07 May 2019 23:34:45 -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=jsXrXBRw+LBgQJhAZNBeeFcQdwpS3eWHXZDSOLy09hg=; b=cx9JD8ZJSjY4QWevc8aSPhLrFwI6XCaZDIQhfbg+f5vspdTHJGFbfTZkExydjs0F3ifHVabl7TZZc3VZKJ9pKVtQny/XwLfNjdRexfLl97p30q9CaESwv7oqJn4gOnVylzsqupgQhrCb74Mj+RVFIJ6SZzuPeDgCQSJRWhb25LuuYNHOLgx3Xuty5UYDhdUUkYiFzbtoJTtNvbIyOIK6gvkzya0Uoag1YnNkZCeCpClTd9xSnyB+de9BylZXFxqz2inqldwsom93T6ZblhvgxRvSP+Q6bcCDsiLZI7AbGOxJyqrxPYAWBpKPBquz4eFsIzX0ba0uKNwd4tYTJpoz9A== Received: from HE1EUR02FT007.eop-EUR02.prod.protection.outlook.com (10.152.10.58) by HE1EUR02HT047.eop-EUR02.prod.protection.outlook.com (10.152.10.179) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384) id 15.20.1856.11; Wed, 8 May 2019 06:34:42 +0000 Received: from AM5PR04MB2993.eurprd04.prod.outlook.com (10.152.10.56) by HE1EUR02FT007.mail.protection.outlook.com (10.152.10.244) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384) id 15.20.1856.11 via Frontend Transport; Wed, 8 May 2019 06:34:42 +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.012; Wed, 8 May 2019 06:34:42 +0000 From: MinnowWare@outlook.com To: "devel@edk2.groups.io" , "jaben.carsey@intel.com" , "Bi, Dandan" , 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 Thread-Topic: [edk2-devel] [patch] ShellBinPkg: Remove ShellBinPkg Thread-Index: AQHU9OiQ239gHckHt0+/VjjL4v4tKqY/8QiAgAA8kICAAO/ygIAH6gWAgAP4FLo= Date: Wed, 8 May 2019 06:34:42 +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:FE1EF8DF6AA650F0FFEA06BC94906D8DC86990E9086A5BA720FB6E7C672E5429;UpperCasedChecksum:5B5BAA76E1DD2BBA576E22BDD27C3415E1920A7F403DC3A9C375A92A9383C1FB;SizeAsReceived:7364;Count:43 x-tmn: [vUvG7FaKa0lpvFo5X8275Io7MqO6V1FG] 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:HE1EUR02HT047; x-ms-traffictypediagnostic: HE1EUR02HT047: x-ms-exchange-purlcount: 7 x-microsoft-antispam-message-info: TK+ADfGeEKekWr2727o5W5zyrGWNc2zJUBCN5cbjsQZ5FsSb44f6YRymWmebLCLIFIfwVjpJ6Suok/Bn4P18DzpDSe6ZPn/u9YYrnYx+vfGWuK0BF9XeT5e0ZI3vBs2NwYB1ErqSOodTD5nSCxzdjwn9SPhJBJ0PYvqgJEQZO4yWqvNo4de0RHejRonxa71a 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: 8a53fee4-d0a2-44de-5cb6-08d6d37f410d X-MS-Exchange-CrossTenant-rms-persistedconsumerorg: 00000000-0000-0000-0000-000000000000 X-MS-Exchange-CrossTenant-originalarrivaltime: 08 May 2019 06:34:42.3956 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Internet X-MS-Exchange-CrossTenant-id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa X-MS-Exchange-Transport-CrossTenantHeadersStamped: HE1EUR02HT047 Content-Language: en-US Content-Type: multipart/alternative; boundary="_000_AM5PR04MB2993F81E0807EF79F45CC9F3C13D0AM5PR04MB2993eurp_" --_000_AM5PR04MB2993F81E0807EF79F45CC9F3C13D0AM5PR04MB2993eurp_ Content-Type: text/plain; charset="Windows-1252" Content-Transfer-Encoding: quoted-printable 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, Jab= en 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 Biesheuvel > > 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, Liming > > 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_AM5PR04MB2993F81E0807EF79F45CC9F3C13D0AM5PR04MB2993eurp_ Content-Type: text/html; charset="Windows-1252" Content-Transfer-Encoding: quoted-printable

From: devel@edk2.groups.= io <devel@edk2.groups.io> on behalf of 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 Laszlo's comment is addressed.
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com>

> -----Original Message-----
> From: Bi, Dandan
> Sent: Wednesday, April 17, 2019 6:12 PM
> To: Laszlo Ersek <lersek@redhat.com>; Ard Biesheuvel
> <ard.biesheuvel@linaro.org>
> Cc: edk2-devel-groups-io <devel@edk2.groups.io>; Carsey, Jaben<= br> > <jaben.carsey@intel.com>; Ni, Ray <ray.ni@intel.com>; Lei= f Lindholm
> <leif.lindholm@linaro.org>; Gao, Liming <liming.gao@intel.co= m>
> 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, J= aben
> > <jaben.carsey@intel.com>; Ni, Ray <ray.ni@intel.com>= ; Leif Lindholm
> > <leif.lindholm@linaro.org>; Gao, Liming <liming.gao@int= el.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><= br> > > >
> > > 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_AM5PR04MB2993F81E0807EF79F45CC9F3C13D0AM5PR04MB2993eurp_--