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 <Bret.Barkelew@microsoft.com>
Sent: Thursday, December 10, 2020 4:09 AM
To: devel@edk2.groups.io; sami.mujawar@arm.com
Cc: Sami Mujawar <sami.mujawar@arm.com>; ard.biesheuvel@arm.com; Yao, Jiewen <jiewen.yao@intel.com>; 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 <sami.mujawar@arm.com>
---
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.<BR>
+# 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)'