From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 134.134.136.126, mailfrom: jaben.carsey@intel.com) Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) by groups.io with SMTP; Fri, 02 Aug 2019 09:18:42 -0700 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga007.fm.intel.com ([10.253.24.52]) by orsmga106.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 02 Aug 2019 09:18:42 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.64,338,1559545200"; d="scan'208";a="174960647" Received: from fmsmsx105.amr.corp.intel.com ([10.18.124.203]) by fmsmga007.fm.intel.com with ESMTP; 02 Aug 2019 09:18:41 -0700 Received: from fmsmsx103.amr.corp.intel.com ([169.254.2.42]) by FMSMSX105.amr.corp.intel.com ([169.254.4.116]) with mapi id 14.03.0439.000; Fri, 2 Aug 2019 09:18:40 -0700 From: "Carsey, Jaben" To: "Gao, Zhichao" , "devel@edk2.groups.io" , "Kinney, Michael D" CC: "Ni, Ray" , Sami Mujawar Subject: Re: [edk2-devel] [Patch v2 2/2] ShellPkg: Add shell with all commands integrated Thread-Topic: [edk2-devel] [Patch v2 2/2] ShellPkg: Add shell with all commands integrated Thread-Index: AQHVSKuJd9VI7TpiPEK32Vwg5L1tuqbnzD4AgAA942A= Date: Fri, 2 Aug 2019 16:18:40 +0000 Message-ID: References: <20190801205541.24792-1-michael.d.kinney@intel.com> <20190801205541.24792-3-michael.d.kinney@intel.com> <3CE959C139B4C44DBEA1810E3AA6F9000B81E444@SHSMSX101.ccr.corp.intel.com> In-Reply-To: <3CE959C139B4C44DBEA1810E3AA6F9000B81E444@SHSMSX101.ccr.corp.intel.com> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiMTY2ZDMyNjktOGU4Ni00ZGI1LWIwOGItNDk2ZDQxMDhhMmI3IiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoibEFwMTJIdFpuZnF2UmxuNFwvcUxmU0hibWZhRUNsbVJuS1dJNmdUUTU1XC8rSXU2VlB3enBvbzkrZXpXdTRlR1VlIn0= x-ctpclassification: CTP_NT dlp-product: dlpe-windows dlp-version: 11.0.600.7 dlp-reaction: no-action x-originating-ip: [10.1.200.107] MIME-Version: 1.0 Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable I thought with this change the DSC would build 2 images. The second image w= ould have the acpiview command and you could ignore that image for your pur= pose. -Jaben > -----Original Message----- > From: Gao, Zhichao > Sent: Thursday, August 1, 2019 10:36 PM > To: devel@edk2.groups.io; Kinney, Michael D > Cc: Carsey, Jaben ; Ni, Ray ; > Sami Mujawar > Subject: RE: [edk2-devel] [Patch v2 2/2] ShellPkg: Add shell with all > commands integrated >=20 > I used to use the shellpkg.dsc to build a shell.efi binary release. > Full configuration of shell is fine. But the acpiview is under developin= g and > the shell spec doesn't contain 'acpiview' yet. > Is it OK to add an additional cmd that the spec isn't mentioned? >=20 > Thanks, > Zhichao >=20 > > -----Original Message----- > > From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of > > Michael D Kinney > > Sent: Friday, August 2, 2019 4:56 AM > > To: devel@edk2.groups.io > > Cc: Carsey, Jaben ; Ni, Ray > > ; Gao, Zhichao ; Sami > Mujawar > > > > Subject: [edk2-devel] [Patch v2 2/2] ShellPkg: Add shell with all > > commands integrated > > > > https://bugzilla.tianocore.org/show_bug.cgi?id=3D1970 > > > > Update ShellPkg DSC file to build an extra version of the Shell with > > all commands integrated. This verifies that the shell can build in > > this max configuration. > > > > Cc: Jaben Carsey > > Cc: Ray Ni > > Cc: Zhichao Gao > > Cc: Sami Mujawar > > Signed-off-by: Michael D Kinney > > --- > > ShellPkg/ShellPkg.dsc | 22 +++++++++++++++++++++- > > 1 file changed, 21 insertions(+), 1 deletion(-) > > > > diff --git a/ShellPkg/ShellPkg.dsc b/ShellPkg/ShellPkg.dsc index > > 6a139b3b91..b2065dc028 100644 > > --- a/ShellPkg/ShellPkg.dsc > > +++ b/ShellPkg/ShellPkg.dsc > > @@ -1,7 +1,7 @@ > > ## @file > > # Shell Package > > # > > -# Copyright (c) 2007 - 2018, Intel Corporation. All rights > > reserved.
> > +# Copyright (c) 2007 - 2019, Intel Corporation. All rights > > +reserved.
> > # Copyright (c) 2018, Arm Limited. All rights reserved.
# > > # SPDX-License-Identifier: BSD-2-Clause-Patent > > @@ -115,6 +115,26 @@ [Components] > > !endif #$(NO_SHELL_PROFILES) > > } > > > > + # > > + # Build a second version of the shell with all commands integrated > > + # ShellPkg/Application/Shell/Shell.inf { > > + > > + FILE_GUID =3D EA4BB293-2D7F-4456-A681-1F22F42CD0BC > > + > > + gEfiShellPkgTokenSpaceGuid.PcdShellLibAutoInitialize|FALSE > > + > > + > > > NULL|ShellPkg/Library/UefiShellLevel2CommandsLib/UefiShellLevel2Comma > > ndsLib.inf > > + > > > NULL|ShellPkg/Library/UefiShellLevel1CommandsLib/UefiShellLevel1Comma > > ndsLib.inf > > + > > > NULL|ShellPkg/Library/UefiShellLevel3CommandsLib/UefiShellLevel3Comma > > ndsLib.inf > > + > > NULL|ShellPkg/Library/UefiShellDriver1CommandsLib/UefiShellDriver1Com > > mandsLib.inf > > + > > NULL|ShellPkg/Library/UefiShellInstall1CommandsLib/UefiShellInstall1Co > > NULL|m > > mandsLib.inf > > + > > > NULL|ShellPkg/Library/UefiShellDebug1CommandsLib/UefiShellDebug1Com > > mandsLib.inf > > + > > NULL|ShellPkg/Library/UefiShellNetwork1CommandsLib/UefiShellNetwork1 > > CommandsLib.inf > > + > > NULL|ShellPkg/Library/UefiShellNetwork2CommandsLib/UefiShellNetwork2 > > CommandsLib.inf > > + > > + > > > NULL|ShellPkg/Library/UefiShellAcpiViewCommandLib/UefiShellAcpiViewCo > > m > > + mandLib.inf > > + } > > + > > > > > ShellPkg/DynamicCommand/TftpDynamicCommand/TftpDynamicCommand. > > inf { > > > > gEfiShellPkgTokenSpaceGuid.PcdShellLibAutoInitialize|FALSE > > -- > > 2.21.0.windows.1 > > > > > >=20