From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 192.55.52.115, mailfrom: liming.gao@intel.com) Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by groups.io with SMTP; Tue, 07 May 2019 23:45:38 -0700 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by fmsmga103.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 07 May 2019 23:45:37 -0700 X-ExtLoop1: 1 Received: from fmsmsx108.amr.corp.intel.com ([10.18.124.206]) by fmsmga004.fm.intel.com with ESMTP; 07 May 2019 23:45:37 -0700 Received: from shsmsx152.ccr.corp.intel.com (10.239.6.52) by FMSMSX108.amr.corp.intel.com (10.18.124.206) with Microsoft SMTP Server (TLS) id 14.3.408.0; Tue, 7 May 2019 23:45:37 -0700 Received: from shsmsx104.ccr.corp.intel.com ([169.254.5.33]) by SHSMSX152.ccr.corp.intel.com ([169.254.6.18]) with mapi id 14.03.0415.000; Wed, 8 May 2019 14:45:34 +0800 From: "Liming Gao" To: Minnow Ware , "devel@edk2.groups.io" , "Carsey, Jaben" , "Bi, Dandan" , Laszlo Ersek , "Ard Biesheuvel" CC: edk2-devel-groups-io , "Ni, Ray" , Leif Lindholm , "Richardson, Brian" Subject: Re: [edk2-devel] [patch] ShellBinPkg: Remove ShellBinPkg Thread-Topic: [edk2-devel] [patch] ShellBinPkg: Remove ShellBinPkg Thread-Index: AQHU9OiNl4yq0klM3k6oJ9Qrp2t1IaY/auyAgAA8kICAAO/ygIAH6gWAgBfe3ACAAId4EA== Date: Wed, 8 May 2019 06:45:34 +0000 Message-ID: <4A89E2EF3DFEDB4C8BFDE51014F606A14E444C2A@SHSMSX104.ccr.corp.intel.com> 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-ctpclassification: CTP_NT x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiYzYxM2Y3NmUtMWZkNi00NmI2LWFhNzgtOTk0ZmZkZWVhNDU5IiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoiSUREell0M1ZQNVNLZVRzbXhDR2JidHQ2SE5RXC9YVDJSMURlYm9wbVU0MW41SkY2NnhyRzBuSXhnbWJqRWVpb0IifQ== dlp-product: dlpe-windows dlp-version: 11.0.600.7 dlp-reaction: no-action x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Return-Path: liming.gao@intel.com Content-Language: en-US Content-Type: multipart/alternative; boundary="_000_4A89E2EF3DFEDB4C8BFDE51014F606A14E444C2ASHSMSX104ccrcor_" --_000_4A89E2EF3DFEDB4C8BFDE51014F606A14E444C2ASHSMSX104ccrcor_ Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Binary Shell can still be got from https://github.com/tianocore/edk2/tree/e= dk2-stable201903/ShellBinPkg/UefiShell/X64 For next stable tag, Binary Shell will be uploaded into https://github.com= /tianocore/edk2/releases tag. Thanks Liming From: Minnow Ware [mailto:minnowware@outlook.com] Sent: Wednesday, May 8, 2019 2: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 "https://github.com/tianocor= e/edk2/raw/master/ShellBinPkg/UefiShell/X64" (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_4A89E2EF3DFEDB4C8BFDE51014F606A14E444C2ASHSMSX104ccrcor_ Content-Type: text/html; charset="us-ascii" Content-Transfer-Encoding: quoted-printable

Binary Shell can stil= l be got from https://github.com/tianocore/edk2/tree/edk2-stable201903/ShellBinPkg/UefiS= hell/X64

 

For next stable tag, = Binary Shell will be uploaded into https://github.com/= tianocore/edk2/releases tag.

 

Thanks

Liming

From: Minnow Ware [mailto:minnowware@outl= ook.com]
Sent: Wednesday, May 8, 2019 2: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

 

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 Laszlo'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_4A89E2EF3DFEDB4C8BFDE51014F606A14E444C2ASHSMSX104ccrcor_--