On 07/28/20 03:11, Zhang, Shenglei wrote: > Hi Laszlo, > >> -----Original Message----- >> From: Laszlo Ersek <ler...@redhat.com> >> Sent: Monday, July 27, 2020 5:51 PM >> To: Zhang, Shenglei <shenglei.zh...@intel.com>; Rebecca Cran >> <rebe...@bsdio.com> >> Cc: devel@edk2.groups.io; Justen, Jordan L <jordan.l.jus...@intel.com>; >> Ard Biesheuvel <ard.biesheu...@arm.com> >> Subject: Re: [PATCH 10/15] OvmfPkg/OvmfPkg.ci.yaml: Add configuration for >> LicenseCheck >> >> On 07/27/20 08:21, Zhang, Shenglei wrote: >>> Hi Laszlo, >>> >>> VbeShim.h is existing in edk2 now. This plugin only checks the patches to >> be checked in. >>> So there's no need to add existing files to this section. >> >> OK, thanks, we can always extend this stanza later, if needed. >> >> Rebecca: once this patch is upstream, please post a separate patch for >> listing >> "OvmfPkg/Bhyve/BhyveRfbDxe/VbeShim.h" in "IgnoreFiles". Otherwise I >> won't be able to merge your patch at >> <https://edk2.groups.io/g/devel/message/62395>. >> >> >> Shenglei: I have a question regarding IgnoreFiles syntax. In >> "MdeModulePkg/MdeModulePkg.ci.yaml", there are two syntaxes: >> >> - The IgnoreFiles stanza for "CharEncodingCheck" uses pathnames that are >> relative to the *project* root: >> >>> ## options defined ci/Plugin/CharEncodingCheck >>> "CharEncodingCheck": { >>> "IgnoreFiles": [ >>> >> "MdeModulePkg/Universal/RegularExpressionDxe/oniguruma/test/testc.c", >>> >> "MdeModulePkg/Universal/RegularExpressionDxe/oniguruma/windows/tes >> tc.c" >>> ] >>> }, >> >> - The IgnoreFiles stanza for "SpellCheck" uses pathnames that are relative to >> the *package* (not project) root: >> >>> "SpellCheck": { >>> ... >>> "IgnoreFiles": [ # use gitignore syntax to ignore >>> errors in matching >> files >>> "Library/LzmaCustomDecompressLib/Sdk/DOC/*" >>> ], >> >> How do we know whether a particular check's IgnoreFiles stanza requires >> project-root-relative or package-root-relative pathnames? > > It depends on the designing of the plugins, likes the check scope. > But looks like all checks' IgnoreFiles stanza only requires > package-root-relative pathnames, currently. > It's recommended to use package-root-relative pathnames because a plugin must > support this format.
Thanks! So we should ignore "Bhyve/BhyveRfbDxe/VbeShim.h". Thanks Laszlo -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#63397): https://edk2.groups.io/g/devel/message/63397 Mute This Topic: https://groups.io/mt/75678218/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-