Good catch Bret. Currently, ArmPlatformPkg.dec is in StandaloneMmCoreEntryPoint.inf. But I don't find any real dependency. ArmSvcLib is in ArmPkg.dec. Hi Sami Is that possible that we just remove ArmPlatformPkg.dec is in StandaloneMmCoreEntryPoint.inf. Thank you Yao Jiewen From: Bret Barkelew Sent: Thursday, December 10, 2020 4:09 AM To: devel@edk2.groups.io; sami.mujawar@arm.com Cc: Sami Mujawar ; ard.biesheuvel@arm.com; Yao, Jiewen ; lersek@redhat.com; thomas.abraham@arm.com; Sughosh.Ganu@arm.com; Matteo.Carlini@arm.com; Ben.Adderson@arm.com; nd@arm.com Subject: RE: [EXTERNAL] [edk2-devel] [PATCH v1 23/25] StandaloneMmPkg: Add EDK2 Core CI support Does StandaloneMm actually need this dependency: ArmPlatformPkg/ArmPlatformPkg.dec - Bret From: Sami Mujawar via groups.io Sent: Wednesday, December 9, 2020 10:49 AM To: devel@edk2.groups.io Cc: Sami Mujawar; ard.biesheuvel@arm.com; Yao, Jiewen; lersek@redhat.com; thomas.abraham@arm.com; Sughosh.Ganu@arm.com; Matteo.Carlini@arm.com; Ben.Adderson@arm.com; nd@arm.com Subject: [EXTERNAL] [edk2-devel] [PATCH v1 23/25] StandaloneMmPkg: Add EDK2 Core CI support The TianoCore EDKII project has introduced a Core CI infrastructure using TianoCore EDKII Tools PIP modules: * https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpypi.org%2Fproject%2Fedk2-pytool-library%2F&data=04%7C01%7Cbret.barkelew%40microsoft.com%7C7cc29e73d78541c28d9608d89c7323f1%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637431365642465867%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=tixOyEmTzX5XCznDFLfapPvPGJ4dMFtQMknQKsivuFE%3D&reserved=0 * https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpypi.org%2Fproject%2Fedk2-pytool-extensions%2F&data=04%7C01%7Cbret.barkelew%40microsoft.com%7C7cc29e73d78541c28d9608d89c7323f1%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637431365642465867%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=50ExwE9RbO44Ojgk5HD0qHeSEheAdmRX%2BG4UnrVUVyE%3D&reserved=0 The edk2\.pytool\Readme.md provides information to configure the environment and to run local builds. This patch defines the necessary settings for enabling the Core CI builds for StandaloneMmPkg. Signed-off-by: Sami Mujawar > --- StandaloneMmPkg/StandaloneMmPkg.ci.yaml | 85 ++++++++++++++++++++ 1 file changed, 85 insertions(+) diff --git a/StandaloneMmPkg/StandaloneMmPkg.ci.yaml b/StandaloneMmPkg/StandaloneMmPkg.ci.yaml new file mode 100644 index 0000000000000000000000000000000000000000..110f461815b11b3775f3e488a46e1a35ed29b4b7 --- /dev/null +++ b/StandaloneMmPkg/StandaloneMmPkg.ci.yaml @@ -0,0 +1,85 @@ +## @file +# CI configuration for StandaloneMmPkg +# +# Copyright (c) 2020, Arm Limited. All rights reserved.
+# SPDX-License-Identifier: BSD-2-Clause-Patent +## +{ + ## options defined .pytool/Plugin/CompilerPlugin + "CompilerPlugin": { + "DscPath": "StandaloneMmPkg.dsc" + }, + + ## options defined .pytool/Plugin/HostUnitTestCompilerPlugin + "HostUnitTestCompilerPlugin": { + "DscPath": "" # Don't support this test + }, + + ## options defined .pytool/Plugin/CharEncodingCheck + "CharEncodingCheck": { + "IgnoreFiles": [] + }, + + ## options defined .pytool/Plugin/DependencyCheck + "DependencyCheck": { + "AcceptableDependencies": [ + "ArmPkg/ArmPkg.dec", + "ArmPlatformPkg/ArmPlatformPkg.dec", + "EmbeddedPkg/EmbeddedPkg.dec", + "StandaloneMmPkg/StandaloneMmPkg.dec", + "MdeModulePkg/MdeModulePkg.dec", + "MdePkg/MdePkg.dec" + ], + # For host based unit tests + "AcceptableDependencies-HOST_APPLICATION":[ + "UnitTestFrameworkPkg/UnitTestFrameworkPkg.dec" + ], + # For UEFI shell based apps + "AcceptableDependencies-UEFI_APPLICATION":[], + "IgnoreInf": [] + }, + + ## options defined .pytool/Plugin/DscCompleteCheck + "DscCompleteCheck": { + "IgnoreInf": [], + "DscPath": "StandaloneMmPkg.dsc" + }, + + ## options defined .pytool/Plugin/HostUnitTestDscCompleteCheck + "HostUnitTestDscCompleteCheck": { + "IgnoreInf": [""], + "DscPath": "" # Don't support this test + }, + + ## options defined .pytool/Plugin/GuidCheck + "GuidCheck": { + "IgnoreGuidName": [], + "IgnoreGuidValue": [], + "IgnoreFoldersAndFiles": [], + "IgnoreDuplicates": [], + }, + + ## options defined .pytool/Plugin/LibraryClassCheck + "LibraryClassCheck": { + "IgnoreHeaderFile": [] + }, + + ## options defined .pytool/Plugin/SpellCheck + "SpellCheck": { + "AuditOnly": False, + "IgnoreFiles": [], # use gitignore syntax to ignore errors + # in matching files + "ExtendWords": [ + "Bsymbolic", + "FwVol", + "mpidr", + "mstrict", + "schedulable", + "StandaloneMMCore", + ], # words to extend to the dictionary for this package + "IgnoreStandardPaths": [], # Standard Plugin defined paths that + # should be ignore + "AdditionalIncludePaths": [] # Additional paths to spell check + # (wildcards supported) + } +} -- 'Guid(CE165669-3EF3-493F-B85D-6190EE5B9759)'