From: "Sean" <spbrogan@outlook.com>
To: devel@edk2.groups.io, pierre.gondois@arm.com, kuqin12@gmail.com
Cc: Sean Brogan <sean.brogan@microsoft.com>,
Bret Barkelew <Bret.Barkelew@microsoft.com>,
Michael D Kinney <michael.d.kinney@intel.com>,
Liming Gao <gaoliming@byosoft.com.cn>,
Laszlo Ersek <lersek@redhat.com>,
Ard Biesheuvel <ardb+tianocore@kernel.org>,
Leif Lindholm <leif@nuviainc.com>,
Sami Mujawar <sami.mujawar@arm.com>,
Jiewen Yao <jiewen.yao@intel.com>,
Supreeth Venkatesh <supreeth.venkatesh@arm.com>
Subject: Re: [edk2-devel] [PATCH v2 0/4] Update Node to 14.x to resolve cspell failure
Date: Tue, 15 Jun 2021 09:04:03 -0700 [thread overview]
Message-ID: <BY3PR19MB490039352441E50737F29221C8309@BY3PR19MB4900.namprd19.prod.outlook.com> (raw)
In-Reply-To: <d34f715f-76c7-3d8a-a636-8de5f4f7b803@arm.com>
Pierre,
Yes we noticed this the other day.
Bugzilla here: https://bugzilla.tianocore.org/show_bug.cgi?id=3454
A patch is on the list https://edk2.groups.io/g/devel/message/76428
it implemented your second option. The good news is this doesn't change
the outcome of overall CI because the Windows Vs2019 Agent should have
caught all issues and since both must pass the mainline should be ok.
There have a been a few issues related to spellchecker that we are
trying to get fixed that has caused missed failures. Hopefully today
the patches will get merged in. But agree it impacts local scenarios
and isn't correct. Thanks again. Nice to see that we both came to the
same conclusion for a fix.
Thanks
Sean
On 6/15/2021 7:29 AM, PierreGondois wrote:
> Hi Kun and Sean,
> As a follow-up to your patch-set, it seems that the spell checker does not currently run as expected. It only runs on files that are 'one level deep' in each package.
> I.e.:
> This was taken from Build/CI_BUILDLOG.txt from the TARGET_ARM_ARMPLATFORM artifact at https://dev.azure.com/tianocore/edk2-ci/_build/results?buildId=23939&view=artifacts&pathAsName=false&type=publishedArtifacts but this can be checked when running the CI locally on ArmPlatformPkg:
>
> -------Log start---------
> INFO - Cmd to run is: cspell --config /home/vsts/work/1/s/Build/ArmPlatformPkg/cspell_actual_config.json ArmPlatformPkg/**/*.c ArmPlatformPkg/**/*.h ArmPlatformPkg/**/*.nasm ArmPlatformPkg/**/*.masm ArmPlatformPkg/**/*.asl ArmPlatformPkg/**/*.dsc ArmPlatformPkg/**/*.dec ArmPlatformPkg/**/*.fdf ArmPlatformPkg/**/*.inf ArmPlatformPkg/**/*.md ArmPlatformPkg/**/*.txt
> INFO - ------------------------------------------------
> INFO - --------------Cmd Output Starting---------------
> INFO - ------------------------------------------------
> INFO - 1/22 ./ArmPlatformPkg/MemoryInitPei/MemoryInitPeiLib.c 829.48ms
> INFO - 2/22 ./ArmPlatformPkg/MemoryInitPei/MemoryInitPeim.c 37.06ms
> INFO - 3/22 ./ArmPlatformPkg/PlatformPei/PlatformPeiLib.c 13.01ms
> INFO - 4/22 ./ArmPlatformPkg/PlatformPei/PlatformPeim.c 16.99ms
> INFO - 5/22 ./ArmPlatformPkg/PrePeiCore/MainMPCore.c 25.68ms
> INFO - 6/22 ./ArmPlatformPkg/PrePeiCore/MainUniCore.c 19.50ms
> INFO - 7/22 ./ArmPlatformPkg/PrePeiCore/PrePeiCore.c 20.86ms
> INFO - 8/22 ./ArmPlatformPkg/PrePi/MainMPCore.c 18.98ms
> INFO - 9/22 ./ArmPlatformPkg/PrePi/MainUniCore.c 9.86ms
> INFO - 10/22 ./ArmPlatformPkg/PrePi/PrePi.c 26.83ms
> INFO - 11/22 ./ArmPlatformPkg/PrePeiCore/PrePeiCore.h 15.71ms
> INFO - 12/22 ./ArmPlatformPkg/PrePi/PrePi.h 16.49ms
> INFO - 13/22 ./ArmPlatformPkg/ArmPlatformPkg.dsc 21.03ms
> INFO - 14/22 ./ArmPlatformPkg/ArmPlatformPkg.dec 24.63ms
> INFO - 15/22 ./ArmPlatformPkg/MemoryInitPei/MemoryInitPeiLib.inf 11.12ms
> INFO - 16/22 ./ArmPlatformPkg/MemoryInitPei/MemoryInitPeim.inf 15.85ms
> INFO - 17/22 ./ArmPlatformPkg/PlatformPei/PlatformPeiLib.inf 9.89ms
> INFO - 18/22 ./ArmPlatformPkg/PlatformPei/PlatformPeim.inf 9.14ms
> INFO - 19/22 ./ArmPlatformPkg/PrePeiCore/PrePeiCoreMPCore.inf 13.97ms
> INFO - 20/22 ./ArmPlatformPkg/PrePeiCore/PrePeiCoreUniCore.inf 9.92ms
> INFO - 21/22 ./ArmPlatformPkg/PrePi/PeiMPCore.inf 12.01ms
> INFO - 22/22 ./ArmPlatformPkg/PrePi/PeiUniCore.inf 18.06ms
> INFO - CSpell: Files checked: 22, Issues found: 0 in 0 files
> INFO - ------------------------------------------------
> INFO - --------------Cmd Output Finished---------------
> INFO - --------- Running Time (mm:ss): 00:01 ----------
> INFO - ----------- Return Code: 0x00000000 ------------
> INFO - ------------------------------------------------
> PROGRESS - --->Test Success: Spell Check Test NO-TARGET
> -------Log end---------
>
> As shown, in the "ArmPlatformPkg/**/*.c" expression, "**" is evaluated as a single directory instead of any number of directories. This can be fixed by:
> - setting the globstar variable (tried on bash): "shopt -s globstar", run "shopt globstar" to check its value, or
> - modifying .pytool/Plugin/SpellCheck/SpellCheck.py to add quotes around the files to check so that cspell evaluates the "**" by itself. This solution seems better as it doesn't require to modify the environment.
>
> Regards,
> Pierre
>
> On 6/14/21 7:34 PM, Kun Qin via groups.io wrote:
>> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3445
>>
>> This patch series is a follow up of previous submission:
>> https://edk2.groups.io/g/devel/message/76419
>>
>> v2 patches mainly focus on feedback for reviewed commits in v1 patches,
>> including:
>> a. Adding "Reviewed-by" tags for applicable patches;
>> b. Adding "Signed-Off-by" tags for myself for all patches;
>>
>> Patch v2 branch: https://github.com/kuqin12/edk2/tree/node_14_v2
>>
>> Cc: Sean Brogan <sean.brogan@microsoft.com>
>> Cc: Bret Barkelew <Bret.Barkelew@microsoft.com>
>> Cc: Michael D Kinney <michael.d.kinney@intel.com>
>> Cc: Liming Gao <gaoliming@byosoft.com.cn>
>> Cc: Laszlo Ersek <lersek@redhat.com>
>> Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
>> Cc: Leif Lindholm <leif@nuviainc.com>
>> Cc: Sami Mujawar <sami.mujawar@arm.com>
>> Cc: Jiewen Yao <jiewen.yao@intel.com>
>> Cc: Supreeth Venkatesh <supreeth.venkatesh@arm.com>
>>
>> Kun Qin (1):
>> Azurepipeline: SpellCheck: Enforce Node dependency to use version 14.x
>>
>> Sean Brogan (3):
>> StandaloneMmPkg: Core: Spelling error in comment
>> ArmPlatformPkg: SpellCheck: Switch spellcheck CI to AuditOnly
>> ArmPkg: SpellCheck: Update valid acronyms in ExtendedWords
>>
>> StandaloneMmPkg/Core/Dispatcher.c | 2 +-
>> .azurepipelines/templates/spell-check-prereq-steps.yml | 2 +-
>> ArmPkg/ArmPkg.ci.yaml | 19 +++++++++++++++++++
>> ArmPlatformPkg/ArmPlatformPkg.ci.yaml | 2 +-
>> 4 files changed, 22 insertions(+), 3 deletions(-)
>>
>
>
>
>
>
next prev parent reply other threads:[~2021-06-15 16:04 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-06-14 18:34 [PATCH v2 0/4] Update Node to 14.x to resolve cspell failure Kun Qin
2021-06-14 18:34 ` [PATCH v2 1/4] StandaloneMmPkg: Core: Spelling error in comment Kun Qin
2021-06-16 14:45 ` Ard Biesheuvel
2021-06-14 18:34 ` [PATCH v2 2/4] ArmPlatformPkg: SpellCheck: Switch spellcheck CI to AuditOnly Kun Qin
2021-06-14 18:34 ` [PATCH v2 3/4] ArmPkg: SpellCheck: Update valid acronyms in ExtendedWords Kun Qin
2021-06-14 18:34 ` [PATCH v2 4/4] Azurepipeline: SpellCheck: Enforce Node dependency to use version 14.x Kun Qin
2021-06-17 22:53 ` [edk2-devel] " Sean
2021-06-15 14:29 ` [edk2-devel] [PATCH v2 0/4] Update Node to 14.x to resolve cspell failure PierreGondois
2021-06-15 16:04 ` Sean [this message]
[not found] ` <168886A01B551E5C.18371@groups.io>
2021-06-16 21:08 ` [edk2-devel] [PATCH v2 4/4] Azurepipeline: SpellCheck: Enforce Node dependency to use version 14.x Kun Qin
2021-06-17 2:05 ` 回复: " gaoliming
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=BY3PR19MB490039352441E50737F29221C8309@BY3PR19MB4900.namprd19.prod.outlook.com \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox