Rebecca: I see Shenglei patch. I plan to finish review early of this week, then merge this patch set, and revert the original change in PatchCheck.py.
Thanks Liming -----Original Message----- From: Rebecca Cran <rebe...@bsdio.com> Sent: 2020年7月25日 9:38 To: devel@edk2.groups.io; Zhang, Shenglei <shenglei.zh...@intel.com> Cc: Kinney, Michael D <michael.d.kin...@intel.com>; Sean Brogan <sean.bro...@microsoft.com>; Bret Barkelew <bret.barke...@microsoft.com>; Dong, Eric <eric.d...@intel.com>; Laszlo Ersek <ler...@redhat.com>; Gao, Zhichao <zhichao....@intel.com>; Yao, Jiewen <jiewen....@intel.com>; Chao Zhang <chao.b.zh...@intel.com>; Justen, Jordan L <jordan.l.jus...@intel.com>; Maciej Rabeda <maciej.rab...@linux.intel.com>; Wu, Jiaxin <jiaxin...@intel.com>; Fu, Siyuan <siyuan...@intel.com>; Gao, Liming <liming....@intel.com>; Wang, Jian J <jian.j.w...@intel.com>; Wu, Hao A <hao.a...@intel.com>; Andrew Fish <af...@apple.com>; Ni, Ray <ray...@intel.com>; Lu, XiaoyuX <xiaoyux...@intel.com>; Ard Biesheuvel <ard.biesheu...@arm.com>; Leif Lindholm <l...@nuviainc.com> Subject: Re: [edk2-devel] [PATCH 00/15] Add a plugin LicenseCheck in open ci On 7/20/20 2:36 AM, Zhang, Shenglei wrote: > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2691 > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2833 > LicenseCheck is now enabled in PatchCheck.py. But there's a patch > "Revert 'BaseTools/PatchCheck.py: Add LicenseCheck'" > to suggest revert the change.These patch series introduce a plugin > LicenseCheck into open ci so that license issues can still be checked > after the checker is disabled in PatchCheck.py. > 1/15 is the plugin implementation. > 2/15 ~ 15/15 introduce sections "IgnoreFiles" to allow developers to > skip license check for some files like generated files. Has there been any progress on this? I'm waiting for it because it's required for my bhyve changes. -- Rebecca Cran -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#63278): https://edk2.groups.io/g/devel/message/63278 Mute This Topic: https://groups.io/mt/75678207/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
