From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) by mx.groups.io with SMTP id smtpd.web12.2278.1572999641134893889 for ; Tue, 05 Nov 2019 16:20:41 -0800 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 134.134.136.126, mailfrom: liming.gao@intel.com) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga002.jf.intel.com ([10.7.209.21]) by orsmga106.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 05 Nov 2019 16:20:40 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.68,271,1569308400"; d="scan'208";a="214074592" Received: from fmsmsx107.amr.corp.intel.com ([10.18.124.205]) by orsmga002.jf.intel.com with ESMTP; 05 Nov 2019 16:20:40 -0800 Received: from fmsmsx602.amr.corp.intel.com (10.18.126.82) by fmsmsx107.amr.corp.intel.com (10.18.124.205) with Microsoft SMTP Server (TLS) id 14.3.439.0; Tue, 5 Nov 2019 16:20:39 -0800 Received: from fmsmsx602.amr.corp.intel.com (10.18.126.82) by fmsmsx602.amr.corp.intel.com (10.18.126.82) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.1713.5; Tue, 5 Nov 2019 16:20:39 -0800 Received: from shsmsx102.ccr.corp.intel.com (10.239.4.154) by fmsmsx602.amr.corp.intel.com (10.18.126.82) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256) id 15.1.1713.5 via Frontend Transport; Tue, 5 Nov 2019 16:20:39 -0800 Received: from shsmsx104.ccr.corp.intel.com ([169.254.5.127]) by shsmsx102.ccr.corp.intel.com ([169.254.2.108]) with mapi id 14.03.0439.000; Wed, 6 Nov 2019 08:20:37 +0800 From: "Liming Gao" To: "Kinney, Michael D" , "devel@edk2.groups.io" , 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: AQHVjpLV+ruXyNuCc0yrXX4gnEzNgqd8Rt9QgAAMKgCAAP3noA== Date: Wed, 6 Nov 2019 00:20:37 +0000 Message-ID: <4A89E2EF3DFEDB4C8BFDE51014F606A14E537E69@SHSMSX104.ccr.corp.intel.com> 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: Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Return-Path: liming.gao@intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Mike: So, github plugin (.azurepipelines, .mergify) uses .yml files. Edk2 pytoo= l plugin uses .yaml. Right? 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 extensi= on > 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 i= s >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