From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail.byosoft.com.cn (mail.byosoft.com.cn [58.240.74.242]) by mx.groups.io with SMTP id smtpd.web12.5181.1611017283480951206 for ; Mon, 18 Jan 2021 16:48:04 -0800 Authentication-Results: mx.groups.io; dkim=missing; spf=none, err=permanent DNS error (domain: byosoft.com.cn, ip: 58.240.74.242, mailfrom: gaoliming@byosoft.com.cn) Received: from DESKTOPS6D0PVI ([58.246.60.130]) (envelope-sender ) by 192.168.6.13 with ESMTP for ; Tue, 19 Jan 2021 08:47:58 +0800 X-WM-Sender: gaoliming@byosoft.com.cn X-WM-AuthFlag: YES X-WM-AuthUser: gaoliming@byosoft.com.cn From: "gaoliming" To: , Cc: "'Chasel Chiu'" , "'Eric Dong'" , "'Kelly Steele'" , "'Michael D Kinney'" , "'Nate DeSimone'" , =?UTF-8?Q?'Philippe_Mathieu-Daud=C3=A9'?= , "'Yi Qian'" , "'Zailiang Sun'" References: <20210118084803.22552-1-lersek@redhat.com> In-Reply-To: <20210118084803.22552-1-lersek@redhat.com> Subject: =?UTF-8?B?5Zue5aSNOiBbZWRrMi1kZXZlbF0gW2VkazItcGxhdGZvcm1zIFBBVENIIHYyXSBJbnRlbCBQbGF0Zm9ybXM6IGFkZCBPcmRlcmVkQ29sbGVjdGlvbkxpYiBjbGFzcyByZXNvbHV0aW9u?= Date: Tue, 19 Jan 2021 08:47:58 +0800 Message-ID: <000401d6edfc$bbaf55e0$330e01a0$@byosoft.com.cn> MIME-Version: 1.0 X-Mailer: Microsoft Outlook 16.0 Thread-Index: AQJaFfBvo/+kB+NT7+uygSQwY+cFlqknzXJQ Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Content-Language: zh-cn Reviewed-by: Liming Gao > -----=E9=82=AE=E4=BB=B6=E5=8E=9F=E4=BB=B6----- > =E5=8F=91=E4=BB=B6=E4=BA=BA: bounce+27952+70497+4905953+8761045@groups.i= o > =E4=BB=A3=E8=A1=A8 Laszlo= Ersek > =E5=8F=91=E9=80=81=E6=97=B6=E9=97=B4: 2021=E5=B9=B41=E6=9C=8818=E6=97=A5= 16:48 > =E6=94=B6=E4=BB=B6=E4=BA=BA: edk2-devel-groups-io > =E6=8A=84=E9=80=81: Chasel Chiu ; Eric Dong > ; Kelly Steele ; Liming Gao > ; Michael D Kinney > ; Nate DeSimone > ; Philippe Mathieu-Daud=C3=A9 > ; Yi Qian ; Zailiang Sun > > =E4=B8=BB=E9=A2=98: [edk2-devel] [edk2-platforms PATCH v2] Intel Platfor= ms: add > OrderedCollectionLib class resolution >=20 > The edk2 patch >=20 > ShellPkg/ShellCommandLib: add ShellSortFileList >=20 > for BZ#3151 will make the >=20 > ShellPkg/Library/UefiShellCommandLib/UefiShellCommandLib.inf >=20 > instance dependent on the OrderedCollectionLib class. Because the shell > binaries in the Intel platform DSCs consume the above UefiShellCommandLi= b > instance, resolve the OrderedCollectionLib class in them. >=20 > Cc: Chasel Chiu > Cc: Eric Dong > Cc: Kelly Steele > Cc: Liming Gao > Cc: Michael D Kinney > Cc: Nate DeSimone > Cc: Philippe Mathieu-Daud=C3=A9 > Cc: Yi Qian > Cc: Zailiang Sun > Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3D3151 > Signed-off-by: Laszlo Ersek > --- >=20 > Notes: > v2: >=20 > - extract OpenBoardPkg update from CometlakeURvp, GalagoPro3, > KabylakeRvp3, BoardX58Ich10, UpXtreme, WhiskeylakeURvp to > "MinPlatformPkg/Include/Dsc/CoreCommonLib.dsc" [Liming] >=20 > - do not pick up Phil's and Nate's R-b, due to the above change >=20 > Platform/Intel/MinPlatformPkg/Include/Dsc/CoreCommonLib.dsc | 1 + > Platform/Intel/QuarkPlatformPkg/Quark.dsc | 1 + > Platform/Intel/QuarkPlatformPkg/QuarkMin.dsc | 1 + > Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgIA32.dsc | 1 + > Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgX64.dsc | 1 + > 5 files changed, 5 insertions(+) >=20 > diff --git a/Platform/Intel/MinPlatformPkg/Include/Dsc/CoreCommonLib.dsc > b/Platform/Intel/MinPlatformPkg/Include/Dsc/CoreCommonLib.dsc > index 5e3405d54785..cb40e111b5dd 100644 > --- a/Platform/Intel/MinPlatformPkg/Include/Dsc/CoreCommonLib.dsc > +++ b/Platform/Intel/MinPlatformPkg/Include/Dsc/CoreCommonLib.dsc > @@ -96,6 +96,7 @@ [LibraryClasses.common] > ShellLib|ShellPkg/Library/UefiShellLib/UefiShellLib.inf > FileHandleLib|MdePkg/Library/UefiFileHandleLib/UefiFileHandleLib.inf > SortLib|MdeModulePkg/Library/UefiSortLib/UefiSortLib.inf > + > OrderedCollectionLib|MdePkg/Library/BaseOrderedCollectionRedBlackTreeLi > b/BaseOrderedCollectionRedBlackTreeLib.inf >=20 > # > # CPU > diff --git a/Platform/Intel/QuarkPlatformPkg/Quark.dsc > b/Platform/Intel/QuarkPlatformPkg/Quark.dsc > index 9f64dcb62724..e29c7465b1e4 100644 > --- a/Platform/Intel/QuarkPlatformPkg/Quark.dsc > +++ b/Platform/Intel/QuarkPlatformPkg/Quark.dsc > @@ -912,6 +912,7 @@ [Components.IA32] >=20 > NULL|ShellPkg/Library/UefiShellInstall1CommandsLib/UefiShellInstall1Com > mandsLib.inf >=20 > NULL|ShellPkg/Library/UefiShellNetwork1CommandsLib/UefiShellNetwork1 > CommandsLib.inf >=20 > HandleParsingLib|ShellPkg/Library/UefiHandleParsingLib/UefiHandleParsing= L > ib.inf > + > OrderedCollectionLib|MdePkg/Library/BaseOrderedCollectionRedBlackTreeLi > b/BaseOrderedCollectionRedBlackTreeLib.inf > PrintLib|MdePkg/Library/BasePrintLib/BasePrintLib.inf >=20 > BcfgCommandLib|ShellPkg/Library/UefiShellBcfgCommandLib/UefiShellBcfgC > ommandLib.inf >=20 > diff --git a/Platform/Intel/QuarkPlatformPkg/QuarkMin.dsc > b/Platform/Intel/QuarkPlatformPkg/QuarkMin.dsc > index db144aab75ae..6fdfba9a58dc 100644 > --- a/Platform/Intel/QuarkPlatformPkg/QuarkMin.dsc > +++ b/Platform/Intel/QuarkPlatformPkg/QuarkMin.dsc > @@ -639,6 +639,7 @@ [Components.IA32] >=20 > NULL|ShellPkg/Library/UefiShellInstall1CommandsLib/UefiShellInstall1Com > mandsLib.inf >=20 > NULL|ShellPkg/Library/UefiShellNetwork1CommandsLib/UefiShellNetwork1 > CommandsLib.inf >=20 > HandleParsingLib|ShellPkg/Library/UefiHandleParsingLib/UefiHandleParsing= L > ib.inf > + > OrderedCollectionLib|MdePkg/Library/BaseOrderedCollectionRedBlackTreeLi > b/BaseOrderedCollectionRedBlackTreeLib.inf > PrintLib|MdePkg/Library/BasePrintLib/BasePrintLib.inf >=20 > BcfgCommandLib|ShellPkg/Library/UefiShellBcfgCommandLib/UefiShellBcfgC > ommandLib.inf >=20 > diff --git a/Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgIA32.dsc > b/Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgIA32.dsc > index 4fa3e2013120..409f31c982d7 100644 > --- a/Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgIA32.dsc > +++ b/Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgIA32.dsc > @@ -1152,6 +1152,7 @@ [Components.IA32] >=20 > NULL|ShellPkg/Library/UefiShellInstall1CommandsLib/UefiShellInstall1Com > mandsLib.inf >=20 > NULL|ShellPkg/Library/UefiShellNetwork1CommandsLib/UefiShellNetwork1 > CommandsLib.inf >=20 > HandleParsingLib|ShellPkg/Library/UefiHandleParsingLib/UefiHandleParsing= L > ib.inf > + > OrderedCollectionLib|MdePkg/Library/BaseOrderedCollectionRedBlackTreeLi > b/BaseOrderedCollectionRedBlackTreeLib.inf > PrintLib|MdePkg/Library/BasePrintLib/BasePrintLib.inf >=20 > BcfgCommandLib|ShellPkg/Library/UefiShellBcfgCommandLib/UefiShellBcfgC > ommandLib.inf > > diff --git a/Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgX64.dsc > b/Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgX64.dsc > index 8adae2e24993..38bd825c8bdc 100644 > --- a/Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgX64.dsc > +++ b/Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgX64.dsc > @@ -1167,6 +1167,7 @@ [Components.X64] >=20 > NULL|ShellPkg/Library/UefiShellInstall1CommandsLib/UefiShellInstall1Com > mandsLib.inf >=20 > NULL|ShellPkg/Library/UefiShellNetwork1CommandsLib/UefiShellNetwork1 > CommandsLib.inf >=20 > HandleParsingLib|ShellPkg/Library/UefiHandleParsingLib/UefiHandleParsing= L > ib.inf > + > OrderedCollectionLib|MdePkg/Library/BaseOrderedCollectionRedBlackTreeLi > b/BaseOrderedCollectionRedBlackTreeLib.inf > PrintLib|MdePkg/Library/BasePrintLib/BasePrintLib.inf >=20 > BcfgCommandLib|ShellPkg/Library/UefiShellBcfgCommandLib/UefiShellBcfgC > ommandLib.inf > >=20 > base-commit: d12d9c963ba3a02252afa9e113c4a3244192ca4a > -- > 2.19.1.3.g30247aa5d201 >=20 >=20 >=20 >=20 >=20