I thought with this change the DSC would build 2 images. The second image would have the acpiview command and you could ignore that image for your purpose.
-Jaben > -----Original Message----- > From: Gao, Zhichao > Sent: Thursday, August 1, 2019 10:36 PM > To: devel@edk2.groups.io; Kinney, Michael D <michael.d.kin...@intel.com> > Cc: Carsey, Jaben <jaben.car...@intel.com>; Ni, Ray <ray...@intel.com>; > Sami Mujawar <sami.muja...@arm.com> > Subject: RE: [edk2-devel] [Patch v2 2/2] ShellPkg: Add shell with all > commands integrated > > 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 developing and > the shell spec doesn't contain 'acpiview' yet. > Is it OK to add an additional cmd that the spec isn't mentioned? > > Thanks, > Zhichao > > > -----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 <jaben.car...@intel.com>; Ni, Ray > > <ray...@intel.com>; Gao, Zhichao <zhichao....@intel.com>; Sami > Mujawar > > <sami.muja...@arm.com> > > Subject: [edk2-devel] [Patch v2 2/2] ShellPkg: Add shell with all > > commands integrated > > > > https://bugzilla.tianocore.org/show_bug.cgi?id=1970 > > > > 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 <jaben.car...@intel.com> > > Cc: Ray Ni <ray...@intel.com> > > Cc: Zhichao Gao <zhichao....@intel.com> > > Cc: Sami Mujawar <sami.muja...@arm.com> > > Signed-off-by: Michael D Kinney <michael.d.kin...@intel.com> > > --- > > 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.<BR> > > +# Copyright (c) 2007 - 2019, Intel Corporation. All rights > > +reserved.<BR> > > # Copyright (c) 2018, Arm Limited. All rights reserved.<BR> # > > # 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 { > > + <Defines> > > + FILE_GUID = EA4BB293-2D7F-4456-A681-1F22F42CD0BC > > + <PcdsFixedAtBuild> > > + gEfiShellPkgTokenSpaceGuid.PcdShellLibAutoInitialize|FALSE > > + <LibraryClasses> > > + > > > 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 { > > <PcdsFixedAtBuild> > > gEfiShellPkgTokenSpaceGuid.PcdShellLibAutoInitialize|FALSE > > -- > > 2.21.0.windows.1 > > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#44862): https://edk2.groups.io/g/devel/message/44862 Mute This Topic: https://groups.io/mt/32683879/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
