REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3443
Current spell check routine on CI build pipelines is ignoring some files incorrectly. The issue is class variable, STANDARD_PLUGIN_DEFINED_PATHS from SpellCheck.py, is a list. In a local function the list is assigned to a local variable, which is then modified based on the package config. But the modification of this local variable will change the class variable and then leaks to the next package. The proposed solution in this patch series is to convert class member to tuple so that is read-only. Then copy into list in the local function before package specific modifications. Patch v1 branch: https://github.com/kuqin12/edk2/tree/spell_check_v1 Cc: Sean Brogan <sean.bro...@microsoft.com> Cc: Bret Barkelew <bret.barke...@microsoft.com> Cc: Michael D Kinney <michael.d.kin...@intel.com> Cc: Liming Gao <gaolim...@byosoft.com.cn> Sean Brogan (1): Pytool: SpellCheck: Fix incorrect file mask across package matrices .pytool/Plugin/SpellCheck/SpellCheck.py | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) -- 2.31.1.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#76309): https://edk2.groups.io/g/devel/message/76309 Mute This Topic: https://groups.io/mt/83435987/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-