Zhichao, I did not modify the build of the standard Shell. It does not include the acpiview.
The patch builds a 2nd version of the shell with a different FILE_GUID for build testing purposes. I think it is very important for the ShellPkg to have build tests for all the content in the ShellPkg to prevent the build failure I saw when I simply wanted to use the acpiview command on an IA32 system. Thanks, Mike > -----Original Message----- > From: Gao, Zhichao > Sent: Thursday, August 1, 2019 10:36 PM > To: [email protected]; Kinney, Michael D > <[email protected]> > Cc: Carsey, Jaben <[email protected]>; Ni, Ray > <[email protected]>; Sami Mujawar <[email protected]> > 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: [email protected] > [mailto:[email protected]] On Behalf Of > > Michael D Kinney > > Sent: Friday, August 2, 2019 4:56 AM > > To: [email protected] > > Cc: Carsey, Jaben <[email protected]>; Ni, Ray > > <[email protected]>; Gao, Zhichao > <[email protected]>; Sami Mujawar > > <[email protected]> > > 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 <[email protected]> > > Cc: Ray Ni <[email protected]> > > Cc: Zhichao Gao <[email protected]> > > Cc: Sami Mujawar <[email protected]> > > Signed-off-by: Michael D Kinney > <[email protected]> > > --- > > 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|FA > LSE > > + <LibraryClasses> > > + > > > NULL|ShellPkg/Library/UefiShellLevel2CommandsLib/UefiSh > ellLevel2Comma > > ndsLib.inf > > + > > > NULL|ShellPkg/Library/UefiShellLevel1CommandsLib/UefiSh > ellLevel1Comma > > ndsLib.inf > > + > > > NULL|ShellPkg/Library/UefiShellLevel3CommandsLib/UefiSh > ellLevel3Comma > > ndsLib.inf > > + > > > NULL|ShellPkg/Library/UefiShellDriver1CommandsLib/UefiS > hellDriver1Com > > mandsLib.inf > > + > > > NULL|ShellPkg/Library/UefiShellInstall1CommandsLib/Uefi > ShellInstall1Co > > NULL|m > > mandsLib.inf > > + > > > NULL|ShellPkg/Library/UefiShellDebug1CommandsLib/UefiSh > ellDebug1Com > > mandsLib.inf > > + > > > NULL|ShellPkg/Library/UefiShellNetwork1CommandsLib/Uefi > ShellNetwork1 > > CommandsLib.inf > > + > > > NULL|ShellPkg/Library/UefiShellNetwork2CommandsLib/Uefi > ShellNetwork2 > > CommandsLib.inf > > + > > + > > > NULL|ShellPkg/Library/UefiShellAcpiViewCommandLib/UefiS > hellAcpiViewCo > > m > > + mandLib.inf > > + } > > + > > > > > ShellPkg/DynamicCommand/TftpDynamicCommand/TftpDynamicC > ommand. > > inf { > > <PcdsFixedAtBuild> > > > gEfiShellPkgTokenSpaceGuid.PcdShellLibAutoInitialize|FA > LSE > > -- > > 2.21.0.windows.1 > > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#44868): https://edk2.groups.io/g/devel/message/44868 Mute This Topic: https://groups.io/mt/32683879/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
