From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by mx.groups.io with SMTP id smtpd.web12.2404.1573000186770965110 for ; Tue, 05 Nov 2019 16:29:46 -0800 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 134.134.136.20, mailfrom: michael.d.kinney@intel.com) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga005.fm.intel.com ([10.253.24.32]) by orsmga101.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 05 Nov 2019 16:29:45 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.68,271,1569308400"; d="scan'208";a="402175164" Received: from orsmsx101.amr.corp.intel.com ([10.22.225.128]) by fmsmga005.fm.intel.com with ESMTP; 05 Nov 2019 16:29:45 -0800 Received: from orsmsx113.amr.corp.intel.com ([169.254.9.28]) by ORSMSX101.amr.corp.intel.com ([169.254.8.212]) with mapi id 14.03.0439.000; Tue, 5 Nov 2019 16:29:45 -0800 From: "Michael D Kinney" To: "Gao, Liming" , "devel@edk2.groups.io" , Sean Brogan , "Kinney, Michael D" 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/OggAECy4D//3xZgA== Date: Wed, 6 Nov 2019 00:29:44 +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> <4A89E2EF3DFEDB4C8BFDE51014F606A14E537E69@SHSMSX104.ccr.corp.intel.com> In-Reply-To: <4A89E2EF3DFEDB4C8BFDE51014F606A14E537E69@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 Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Yes. Mike > -----Original Message----- > From: Gao, Liming > Sent: Tuesday, November 5, 2019 4:21 PM > To: Kinney, Michael D ; > devel@edk2.groups.io; Sean Brogan > > Subject: RE: [Patch v3 12/22] MdePkg: Add YAML file for > CI builds >=20 > Mike: > So, github plugin (.azurepipelines, .mergify) uses > .yml files. Edk2 pytool plugin uses .yaml. Right? >=20 > Thanks > Liming > >-----Original Message----- > >From: Kinney, Michael D > >Sent: Wednesday, November 06, 2019 1:05 AM > >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 > > > >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 > > 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 extension > > 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/CharEncodingCheck > >https://github.com/tianocore/edk2-staging/tree/edk2- > >ci/.pytool/Plugin/CompilerPlugin > >https://github.com/tianocore/edk2-staging/tree/edk2- > >ci/.pytool/Plugin/DependencyCheck > >https://github.com/tianocore/edk2-staging/tree/edk2- > >ci/.pytool/Plugin/DscCompleteCheck > >https://github.com/tianocore/edk2-staging/tree/edk2- > >ci/.pytool/Plugin/GuidCheck > >https://github.com/tianocore/edk2-staging/tree/edk2- > >ci/.pytool/Plugin/LibraryClassCheck > >https://github.com/tianocore/edk2-staging/tree/edk2- > >ci/.pytool/Plugin/SpellCheck > > > >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 > >> > >> Mike: > >> Is there some basic introduction for YAML file > syntax? > >> > >> 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