From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by mx.groups.io with SMTP id smtpd.web08.9044.1619533831889900664 for ; Tue, 27 Apr 2021 07:30:32 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: arm.com, ip: 217.140.110.172, mailfrom: pierre.gondois@arm.com) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id DB1B5ED1; Tue, 27 Apr 2021 07:30:30 -0700 (PDT) Received: from [10.57.3.114] (unknown [10.57.3.114]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id E0B133F73B; Tue, 27 Apr 2021 07:30:29 -0700 (PDT) Subject: Re: [PATCH v1 06/12] ArmPkg: Add ArmPkg.ci.yaml To: Sami Mujawar , "devel@edk2.groups.io" , nd References: <20210421122048.11729-1-Pierre.Gondois@arm.com> <20210421122048.11729-7-Pierre.Gondois@arm.com> From: "PierreGondois" Message-ID: Date: Tue, 27 Apr 2021 15:30:26 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.10.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=windows-1252; format=flowed Content-Language: en-US Content-Transfer-Encoding: quoted-printable Hi Sami, I think the lists are already alphabetically ordered. I will re-order=20 the libraries in ArmPkg.dec though. I will remove the "eio'ed" word from the exception list in a V2. Thanks for the review, Pierre On 4/22/21 11:02 AM, Sami Mujawar wrote: > > Hi Pierre, > > I have a few minor comments marked inline as [SAMI]. > > With those changed. > > Reviewed-by: Sami Mujawar > > Regards, > > Sami Mujawar > > *From: *Pierre.Gondois@arm.com > *Date: *Wednesday, 21 April 2021 at 13:21 > *To: *devel@edk2.groups.io , Sami Mujawar=20 > , leif@nuviainc.com ,=20 > ardb+tianocore@kernel.org ,=20 > sean.brogan@microsoft.com ,=20 > Bret.Barkelew@microsoft.com > *Subject: *[PATCH v1 06/12] ArmPkg: Add ArmPkg.ci.yaml > > From: Pierre Gondois > > Add ArmPkg.ci.yaml to configure the CI for the > ArmPkg. > > Signed-off-by: Pierre Gondois > --- > =A0ArmPkg/ArmPkg.ci.yaml | 222 ++++++++++++++++++++++++++++++++++++++++= ++ > =A01 file changed, 222 insertions(+) > =A0create mode 100644 ArmPkg/ArmPkg.ci.yaml > > diff --git a/ArmPkg/ArmPkg.ci.yaml b/ArmPkg/ArmPkg.ci.yaml > new file mode 100644 > index 000000000000..ba502cd647c9 > --- /dev/null > +++ b/ArmPkg/ArmPkg.ci.yaml > @@ -0,0 +1,222 @@ > +## @file > +# CI configuration for ArmPkg > +# > +# Copyright (c) 2021, Arm Limited. All rights reserved.
> +# SPDX-License-Identifier: BSD-2-Clause-Patent > +## > +{ > +=A0=A0=A0 ## options defined .pytool/Plugin/LicenseCheck > +=A0=A0=A0 "LicenseCheck": { > +=A0=A0=A0=A0=A0=A0=A0 "IgnoreFiles": [] > +=A0=A0=A0 }, > + > +=A0=A0=A0 "EccCheck": { > +=A0=A0=A0=A0=A0=A0=A0 ## Exception sample looks like below: > +=A0=A0=A0=A0=A0=A0=A0 ## "ExceptionList": [ > +=A0=A0=A0=A0=A0=A0=A0 ##=A0=A0=A0=A0 "", "" > +=A0=A0=A0=A0=A0=A0=A0 ## ] > +=A0=A0=A0=A0=A0=A0=A0 "ExceptionList": [ > +=A0=A0=A0=A0=A0=A0=A0 ], > +=A0=A0=A0=A0=A0=A0=A0 ## Both file path and directory path are accepte= d. > +=A0=A0=A0=A0=A0=A0=A0 "IgnoreFiles": [ > +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 "Library/ArmSoftFloatLib/berkeley-so= ftfloat-3" > +=A0=A0=A0=A0=A0=A0=A0 ] > +=A0=A0=A0 }, > + > +=A0=A0=A0 ## options defined .pytool/Plugin/CompilerPlugin > +=A0=A0=A0 "CompilerPlugin": { > +=A0=A0=A0=A0=A0=A0=A0 "DscPath": "ArmPkg.dsc" > +=A0=A0=A0 }, > + > +=A0=A0=A0 ## options defined .pytool/Plugin/HostUnitTestCompilerPlugin > +=A0=A0=A0 "HostUnitTestCompilerPlugin": { > +=A0=A0=A0=A0=A0=A0=A0 "DscPath": "" # Don't support this test > +=A0=A0=A0 }, > + > +=A0=A0=A0 ## options defined .pytool/Plugin/CharEncodingCheck > +=A0=A0=A0 "CharEncodingCheck": { > +=A0=A0=A0=A0=A0=A0=A0 "IgnoreFiles": [] > +=A0=A0=A0 }, > + > +=A0=A0=A0 ## options defined .pytool/Plugin/DependencyCheck > +=A0=A0=A0 "DependencyCheck": { > +=A0=A0=A0=A0=A0=A0=A0 "AcceptableDependencies": [ > +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 "ArmPlatformPkg/ArmPlatformPkg.dec", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 "ArmPkg/ArmPkg.dec", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 "EmbeddedPkg/EmbeddedPkg.dec", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 "MdeModulePkg/MdeModulePkg.dec", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 "MdePkg/MdePkg.dec", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 "ShellPkg/ShellPkg.dec" > > [SAMI] Can this list be sorted in alphabetical order, please? > > [/SAMI] > +=A0=A0=A0=A0=A0=A0=A0 ], > +=A0=A0=A0=A0=A0=A0=A0 # For host based unit tests > +=A0=A0=A0=A0=A0=A0=A0 "AcceptableDependencies-HOST_APPLICATION":[ > +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 "UnitTestFrameworkPkg/UnitTestFramew= orkPkg.dec" > +=A0=A0=A0=A0=A0=A0=A0 ], > +=A0=A0=A0=A0=A0=A0=A0 # For UEFI shell based apps > +=A0=A0=A0=A0=A0=A0=A0 "AcceptableDependencies-UEFI_APPLICATION":[], > +=A0=A0=A0=A0=A0=A0=A0 "IgnoreInf": [] > +=A0=A0=A0 }, > + > +=A0=A0=A0 ## options defined .pytool/Plugin/DscCompleteCheck > +=A0=A0=A0 "DscCompleteCheck": { > +=A0=A0=A0=A0=A0=A0=A0 "IgnoreInf": [], > +=A0=A0=A0=A0=A0=A0=A0 "DscPath": "ArmPkg.dsc" > +=A0=A0=A0 }, > + > +=A0=A0=A0 ## options defined .pytool/Plugin/HostUnitTestDscCompleteChe= ck > +=A0=A0=A0 "HostUnitTestDscCompleteCheck": { > +=A0=A0=A0=A0=A0=A0=A0 "IgnoreInf": [""], > +=A0=A0=A0=A0=A0=A0=A0 "DscPath": "" # Don't support this test > +=A0=A0=A0 }, > + > +=A0=A0=A0 ## options defined .pytool/Plugin/GuidCheck > +=A0=A0=A0 "GuidCheck": { > +=A0=A0=A0=A0=A0=A0=A0 "IgnoreGuidName": [], > +=A0=A0=A0=A0=A0=A0=A0 "IgnoreGuidValue": [], > +=A0=A0=A0=A0=A0=A0=A0 "IgnoreFoldersAndFiles": [], > +=A0=A0=A0=A0=A0=A0=A0 "IgnoreDuplicates": [], > +=A0=A0=A0 }, > + > +=A0=A0=A0 ## options defined .pytool/Plugin/LibraryClassCheck > +=A0=A0=A0 "LibraryClassCheck": { > +=A0=A0=A0=A0=A0=A0=A0 "IgnoreHeaderFile": [] > +=A0=A0=A0 }, > + > +=A0=A0=A0 ## options defined .pytool/Plugin/SpellCheck > +=A0=A0=A0 "SpellCheck": { > +=A0=A0=A0=A0=A0=A0=A0 "AuditOnly": False, > +=A0=A0=A0=A0=A0=A0=A0 "IgnoreFiles": [ > + "Library/ArmSoftFloatLib/berkeley-softfloat-3/**" > +=A0=A0=A0=A0=A0=A0=A0 ],=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0= =A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 # use gitignore syntax to ignore=20 > errors > +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0= =A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 # in matching files > +=A0=A0=A0=A0=A0=A0=A0 "ExtendWords": [ > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "api's", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "ackintid", > > [SAMI] Can this list be sorted in alphabetical order, please? > > [/SAMI] > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "actlr", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "aeabi", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "ashldi", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "ashrdi", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "ccidx", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "ccsidr", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "clidr", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "clrex", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "clzsi", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "cpuactlr", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "csselr", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "ctzsi", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "cygdrive", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "cygpaths", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "datas", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "dcmpeq", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "dcmpge", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "dcmpgt", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "dcmple", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "dcmplt", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "ddisable", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "divdi", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "divsi", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "dmdepkg", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "drsub", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "eoi'ed", > > [SAMI] I don=92t think there is such a word. Should the original text b= e=20 > fixed? > > [/SAMI] > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "fcmpeq", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "fcmpge", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "fcmpgt", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "fcmple", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "fcmplt", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "ffreestanding", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "frsub", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "hisilicon", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "iccbpr", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "icciar", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "iccicr", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "icciidr", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "iccpmr", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "icdicer", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "icdicfr", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "icdictr", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "icdiser", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "icdisr", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "icdsgir", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "icenabler", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "intid", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "ipriority", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "irouter", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "isenabler", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "istatus", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "itargets", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "lable", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "ldivmod", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "ldmdb", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "ldmia", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "ldrbt", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "ldrex", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "ldrexb", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "ldrexd", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "ldrexh", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "ldrhbt", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "ldrht", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "ldrsb", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "ldrsbt", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "ldrsh", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "lshrdi", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "moddi", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "modsi", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "mpidr", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "muldi", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "mullu", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "nonshareable", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "nsacr", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "nsasedis", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "nuvia", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "oldit", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "readc", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "revsh", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "rfedb", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "sctlr", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "smccc", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "smlabb", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "smlabt", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "smlad", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "smladx", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "smlatb", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "smlatt", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "smlawb", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "smlawt", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "smlsd", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "smlsdx", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "smmla", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "smmlar", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "smmls", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "smmlsr", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "sourcery", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "srsdb", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "stmdb", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "stmia", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "strbt", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "strexb", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "strexd", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "strexh", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "strht", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "switchu", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "tpidrurw", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "ttbcr", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "typer", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "ucmpdi", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "udivdi", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "udivmoddi", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "udivsi", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "uefi's", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "uldiv", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "umoddi", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "umodsi", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "usada", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "vlpis", > +=A0=A0=A0=A0=A0=A0=A0=A0=A0 "writec" > +=A0=A0=A0=A0=A0=A0=A0 ],=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0= =A0=A0=A0=A0=A0=A0=A0=A0=A0 # words to extend to the=20 > dictionary for this package > +=A0=A0=A0=A0=A0=A0=A0 "IgnoreStandardPaths": [=A0=A0=A0 # Standard Plu= gin defined paths that > +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 "*.asm", "*.s"=A0=A0=A0=A0=A0=A0=A0=A0= =A0 # should be ignore > +=A0=A0=A0=A0=A0=A0=A0 ], > +=A0=A0=A0=A0=A0=A0=A0 "AdditionalIncludePaths": [] # Additional paths = to spell check > +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0= =A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 # (wildcards supported) > +=A0=A0=A0 } > +} > --=20 > 2.17.1 >