Hi Bret,

 

I think the reference to ArmPlatformPkg.dec can be removed from StandaloneMmCoreEntryPoint.inf which should fix the following error and also remove the dependency.

 

ERROR - Dependency Check: Invalid Dependency INF: StandaloneMmPkg/Library/StandaloneMmCoreEntryPoint/StandaloneMmCoreEntryPoint.inf depends on pkg ArmPlatformPkg/ArmPlatformPkg.dec

 

I will send an updated patch with this fixed.

 

Regards,

 

Sami Mujawar

From: Bret Barkelew <Bret.Barkelew@microsoft.com>
Sent: 09 December 2020 08:09 PM
To: devel@edk2.groups.io; Sami Mujawar <Sami.Mujawar@arm.com>
Cc: Sami Mujawar <Sami.Mujawar@arm.com>; Ard Biesheuvel <Ard.Biesheuvel@arm.com>; Yao, Jiewen <jiewen.yao@intel.com>; lersek@redhat.com; Thomas Abraham <thomas.abraham@arm.com>; Sughosh Ganu <Sughosh.Ganu@arm.com>; Matteo Carlini <Matteo.Carlini@arm.com>; Ben Adderson <Ben.Adderson@arm.com>; nd <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&amp;data=04%7C01%7Cbret.barkelew%40microsoft.com%7C7cc29e73d78541c28d9608d89c7323f1%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637431365642465867%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&amp;sdata=tixOyEmTzX5XCznDFLfapPvPGJ4dMFtQMknQKsivuFE%3D&amp;reserved=0
  *
https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpypi.org%2Fproject%2Fedk2-pytool-extensions%2F&amp;data=04%7C01%7Cbret.barkelew%40microsoft.com%7C7cc29e73d78541c28d9608d89c7323f1%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637431365642465867%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&amp;sdata=50ExwE9RbO44Ojgk5HD0qHeSEheAdmRX%2BG4UnrVUVyE%3D&amp;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)'