From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by mx.groups.io with SMTP id smtpd.web12.6444.1572973498608579124 for ; Tue, 05 Nov 2019 09:04:58 -0800 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 134.134.136.65, mailfrom: michael.d.kinney@intel.com) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga006.fm.intel.com ([10.253.24.20]) by orsmga103.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 05 Nov 2019 09:04:57 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.68,271,1569308400"; d="scan'208";a="403419437" Received: from orsmsx107.amr.corp.intel.com ([10.22.240.5]) by fmsmga006.fm.intel.com with ESMTP; 05 Nov 2019 09:04:57 -0800 Received: from orsmsx158.amr.corp.intel.com (10.22.240.20) by ORSMSX107.amr.corp.intel.com (10.22.240.5) with Microsoft SMTP Server (TLS) id 14.3.439.0; Tue, 5 Nov 2019 09:04:57 -0800 Received: from orsmsx113.amr.corp.intel.com ([169.254.9.28]) by ORSMSX158.amr.corp.intel.com ([169.254.10.56]) with mapi id 14.03.0439.000; Tue, 5 Nov 2019 09:04:56 -0800 From: "Michael D Kinney" To: "Gao, Liming" , "devel@edk2.groups.io" , "Kinney, Michael D" , Sean Brogan Subject: Re: [Patch v3 12/22] MdePkg: Add YAML file for CI builds Thread-Topic: [Patch v3 12/22] MdePkg: Add YAML file for CI builds Thread-Index: AQHVk7IicK3T/jqAQE+vvUq37Vrg+Kd8y/Og Date: Tue, 5 Nov 2019 17:04:56 +0000 Message-ID: References: <20191029195517.20028-1-michael.d.kinney@intel.com> <20191029195517.20028-13-michael.d.kinney@intel.com> <4A89E2EF3DFEDB4C8BFDE51014F606A14E5379F6@SHSMSX104.ccr.corp.intel.com> In-Reply-To: <4A89E2EF3DFEDB4C8BFDE51014F606A14E5379F6@SHSMSX104.ccr.corp.intel.com> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-product: dlpe-windows dlp-version: 11.2.0.6 dlp-reaction: no-action x-originating-ip: [10.22.254.139] MIME-Version: 1.0 Return-Path: michael.d.kinney@intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Hi Liming, There are two types of YAML files in this patch series. 1) Azure Piplelines .yml files. https://docs.microsoft.com/en-us/azure/devops/pipelines/yaml-schema?view= =3Dazure-devops&tabs=3Dschema 2) EDK II Package .yaml files. These specify the set of pytool extensions = to=20 use to make sure an EDK II package passes all the required checks. The contents of these files are JSON format and the syntax for each extensio= n=20 is defined by each pytool plugin. JSON is considered a subset of YAML. https://github.com/tianocore/edk2-staging/tree/edk2-ci/.pytool https://github.com/tianocore/edk2-staging/tree/edk2-ci/.pytool/Plugin/CharE= ncodingCheck https://github.com/tianocore/edk2-staging/tree/edk2-ci/.pytool/Plugin/Compi= lerPlugin https://github.com/tianocore/edk2-staging/tree/edk2-ci/.pytool/Plugin/Depen= dencyCheck https://github.com/tianocore/edk2-staging/tree/edk2-ci/.pytool/Plugin/DscCo= mpleteCheck https://github.com/tianocore/edk2-staging/tree/edk2-ci/.pytool/Plugin/GuidC= heck https://github.com/tianocore/edk2-staging/tree/edk2-ci/.pytool/Plugin/Libra= ryClassCheck https://github.com/tianocore/edk2-staging/tree/edk2-ci/.pytool/Plugin/Spell= Check Please let us know if the plugin documentation including the YAML syntax is= not clear. Best regards, Mike > -----Original Message----- > From: Gao, Liming > Sent: Tuesday, November 5, 2019 12:22 AM > To: Kinney, Michael D ; > devel@edk2.groups.io > Subject: RE: [Patch v3 12/22] MdePkg: Add YAML file for > CI builds >=20 > Mike: > Is there some basic introduction for YAML file > syntax? >=20 > Thanks > Liming > >-----Original Message----- > >From: Kinney, Michael D > >Sent: Wednesday, October 30, 2019 3:55 AM > >To: devel@edk2.groups.io > >Cc: Gao, Liming > >Subject: [Patch v3 12/22] MdePkg: Add YAML file for CI > builds > > > >https://bugzilla.tianocore.org/show_bug.cgi?id=3D2315 > > > >Add YAML file to the package directory with the > configuration of the > >checks to perform during a CI build. > > > >Cc: Liming Gao > >Signed-off-by: Michael D Kinney > > >--- > > MdePkg/MdePkg.ci.yaml | 90 > >+++++++++++++++++++++++++++++++++++++++++++ > > 1 file changed, 90 insertions(+) > > create mode 100644 MdePkg/MdePkg.ci.yaml > > > >diff --git a/MdePkg/MdePkg.ci.yaml > b/MdePkg/MdePkg.ci.yaml new file > >mode 100644 index 0000000000..4015c07f16 > >--- /dev/null > >+++ b/MdePkg/MdePkg.ci.yaml > >@@ -0,0 +1,90 @@ > >+## > >+# CI configuration for MdePkg > >+# > >+# Copyright (c) Microsoft Corporation > >+# SPDX-License-Identifier: BSD-2-Clause-Patent ## { > >+ ## options defined ci/Plugin/CompilerPlugin > >+ "CompilerPlugin": { > >+ "DscPath": "MdePkg.dsc" > >+ }, > >+ > >+ ## options defined ci/Plugin/CharEncodingCheck > >+"CharEncodingCheck": { > >+ "IgnoreFiles": [] > >+ }, > >+ > >+ ## options defined ci/Plugin/DependencyCheck > >+ "DependencyCheck": { > >+ "AcceptableDependencies": [ > >+ "MdePkg/MdePkg.dec" > >+ ], > >+ # For host based unit tests > >+ "AcceptableDependencies-HOST_APPLICATION":[], > >+ # For UEFI shell based apps > >+ "AcceptableDependencies-UEFI_APPLICATION":[], > >+ "IgnoreInf": [] > >+ }, > >+ > >+ ## options defined ci/Plugin/DscCompleteCheck > >+ "DscCompleteCheck": { > >+ "IgnoreInf": [], > >+ "DscPath": "MdePkg.dsc" > >+ }, > >+ > >+ ## options defined ci/Plugin/GuidCheck > >+ "GuidCheck": { > >+ "IgnoreGuidName": [ > >+ "gEfiFirmwareVolumeTopFileGuid" # sec > modules must be set > >+ to this > >guid > >+ ], > >+ "IgnoreGuidValue": ["00000000-0000-0000-0000- > 000000000000"], > >+ "IgnoreFoldersAndFiles": [], > >+ "IgnoreDuplicates": [ > >+ > >"gEfiFirmwareVolumeBlockProtocolGuid=3DgEfiFirmwareVolume > Block2Protocol > >Guid", > >+ > "gEfiMmAccessProtocolGuid=3DgEfiSmmAccess2ProtocolGuid", > >+ > >"gEfiSmmCommunicationProtocolGuid=3DgEfiMmCommunicationPr > otocolGuid", > >+ > >"gEfiMmPciRootBridgeIoProtocolGuid=3DgEfiSmmPciRootBridge > IoProtocolGuid", > >+ > "gEfiTrEEProtocolGuid=3DgEfiTcg2ProtocolGuid", > >+ > >"gEfiSmmPowerButtonDispatch2ProtocolGuid=3DgEfiMmPowerBut > tonDispatch > >ProtocolGuid", > >+ > "gEfiSmmBase2ProtocolGuid=3DgEfiMmBaseProtocolGuid", > >+ > >"gEfiSmmUsbDispatch2ProtocolGuid=3DgEfiMmUsbDispatchProto > colGuid", > >+ > "gEfiSmmCpuProtocolGuid=3DgEfiMmCpuProtocolGuid", > >+ "gEfiAcpiTableGuid=3DgEfiAcpi20TableGuid", > >+ > "gEfiSmmControl2ProtocolGuid=3DgEfiMmControlProtocolGuid", > >+ > >"gEfiSmmStandbyButtonDispatch2ProtocolGuid=3DgEfiMmStandb > yButtonDispat > >chProtocolGuid", > >+ > >+ > "gEfiSmmStatusCodeProtocolGuid=3DgEfiMmStatusCodeProtocolG > uid", > >+ > >"gEfiDxeSmmReadyToLockProtocolGuid=3DgEfiDxeMmReadyToLock > ProtocolGui > >d", > >+ > >"gEfiSmmIoTrapDispatch2ProtocolGuid=3DgEfiMmIoTrapDispatc > hProtocolGuid", > >+ > >"gEfiSmmReadyToLockProtocolGuid=3DgEfiMmReadyToLockProtoc > olGuid", > >+ > "gEfiSmmSxDispatch2ProtocolGuid=3DgEfiMmSxDispatchProtocol > Guid", > >+ > "gEfiPeiCapsulePpiGuid=3DgPeiCapsulePpiGuid", > >+ > "gEfiSmmCpuIo2ProtocolGuid=3DgEfiMmCpuIoProtocolGuid", > >+ > >+ > "gEfiSmmRscHandlerProtocolGuid=3DgEfiMmRscHandlerProtocolG > uid", > >+ > >"gEfiSmmConfigurationProtocolGuid=3DgEfiMmConfigurationPr > otocolGuid", > >+ > >"gEfiSmmGpiDispatch2ProtocolGuid=3DgEfiMmGpiDispatchProto > colGuid", > >+ > "gEfiSmmEndOfDxeProtocolGuid=3DgEfiMmEndOfDxeProtocolGuid" > , > >+ > >+ > "gEfiSmmSwDispatch2ProtocolGuid=3DgEfiMmSwDispatchProtocol > Guid", > >+ > >"gEfiDebugPortProtocolGuid=3DgEfiDebugPortVariableGuid=3DgE > fiDebugPortDevi > >cePathGuid", ## is this a bug > >+ > >"gEfiProcessorSpecificErrorSectionGuid=3DgEfiIa32X64Proce > ssorErrorSection > >Gui > >d", ## is this a bug > >+ > >"gEfiSmmPeriodicTimerDispatch2ProtocolGuid=3DgEfiMmPeriod > icTimerDispatch > >ProtocolGuid", > >+ > "gEdkiiFormBrowserExProtocolGuid=3DgEfiFormBrowserExProtoc > olGuid", > >+ > "gEfiPeiMmAccessPpiGuid=3DgPeiSmmAccessPpiGuid", > >+ > "gPeiSmmControlPpiGuid=3DgEfiPeiMmControlPpiGuid" > >+ ] > >+ }, > >+ > >+ ## options defined ci/Plugin/LibraryClassCheck > >+ "LibraryClassCheck": { > >+ "IgnoreHeaderFile": [] > >+ }, > >+ > >+ ## options defined ci/Plugin/SpellCheck > >+ "SpellCheck": { > >+ "AuditOnly": True, # Fails test but > run in AuditOnly mode to collect > >log > >+ "IgnoreFiles": [], # use gitignore > syntax to ignore errors in matching > >files > >+ "ExtendWords": [], # 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) > >+ } > >+} > >-- > >2.21.0.windows.1