From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by mx.groups.io with SMTP id smtpd.web12.4741.1596025694698132285 for ; Wed, 29 Jul 2020 05:28:14 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 192.55.52.115, mailfrom: shenglei.zhang@intel.com) IronPort-SDR: Xw1fsMiUyUTClLOIvnLpkn7rtQwpt3tncY9uq2hpAtnfkreQnw28oVoBWJVciK0ITb1uSbe+zF nodrZTsFbayQ== X-IronPort-AV: E=McAfee;i="6000,8403,9696"; a="150571072" X-IronPort-AV: E=Sophos;i="5.75,410,1589266800"; d="scan'208";a="150571072" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga008.fm.intel.com ([10.253.24.58]) by fmsmga103.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 29 Jul 2020 05:28:13 -0700 IronPort-SDR: Kz/jnP5yr8g9/dCXHDclHuoU8pszI+vNNrYYafkl9UNAC9dZZ0H8y+oSZzUWVMg8AiJ6R4dpon oYdB45UYyJXA== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.75,410,1589266800"; d="scan'208";a="273872523" Received: from shenglei-dev.ccr.corp.intel.com ([10.239.154.36]) by fmsmga008.fm.intel.com with ESMTP; 29 Jul 2020 05:28:08 -0700 From: "Zhang, Shenglei" To: devel@edk2.groups.io Cc: Michael D Kinney , Sean Brogan , Bret Barkelew , Eric Dong , Laszlo Ersek , Zhichao Gao , Jiewen Yao , Chao Zhang , Jordan Justen , Maciej Rabeda , Jiaxin Wu , Siyuan Fu , Liming Gao , Jian J Wang , Hao A Wu , Andrew Fish , Ray Ni , Xiaoyu Lu , Ard Biesheuvel , Leif Lindholm Subject: [PATCHv v2 00/15] Add a plugin LicenseCheck in open ci Date: Wed, 29 Jul 2020 20:28:05 +0800 Message-Id: <20200729122806.21304-1-shenglei.zhang@intel.com> X-Mailer: git-send-email 2.18.0.windows.1 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. Only BSD-2-Clause-Patent can pass this checker. v2: BSD-3-Clause-Patent can not pass this checker in v2 patch. * 2/15~15/15 have been given reviewed-by. To avoid making noise, I only send out cover letter and 1/15. Cc: Michael D Kinney Cc: Sean Brogan Cc: Bret Barkelew Cc: Eric Dong Cc: Laszlo Ersek Cc: Zhichao Gao Cc: Jiewen Yao Cc: Chao Zhang Cc: Jordan Justen Cc: Maciej Rabeda Cc: Jiaxin Wu Cc: Siyuan Fu Cc: Liming Gao Cc: Jian J Wang Cc: Hao A Wu Cc: Jordan Justen Cc: Andrew Fish Cc: Ray Ni Cc: Jian J Wang Cc: Xiaoyu Lu Cc: Ard Biesheuvel Cc: Leif Lindholm Shenglei Zhang (15): .pytool/Plugin: Add a plugin LicenseCheck FatPkg/FatPkg.ci.yaml: Add configuration for LicenseCheck ArmVirtPkg/ArmVirtPkg.ci.yaml: Add configuration for LicenseCheck CryptoPkg/CryptoPkg.ci.yaml: Add configuration for LicenseCheck EmulatorPkg/EmulatorPkg.ci.yaml: Add configuration for LicenseCheck FmpDevicePkg/FmpDevicePkg.ci.yaml: Add configuration for LicenseCheck MdeModulePkg/MdeModulePkg.ci.yaml: Add configuration for LicenseCheck MdePkg/MdePkg.ci.yaml: Add configuration for LicenseCheck NetworkPkg/NetworkPkg.ci.yaml: Add configuration for LicenseCheck OvmfPkg/OvmfPkg.ci.yaml: Add configuration for LicenseCheck PcAtChipsetPkg/PcAtChipsetPkg.ci.yaml: Add configuration for LicenseCheck SecurityPkg/SecurityPkg.ci.yaml: Add configuration for LicenseCheck ShellPkg/ShellPkg.ci.yaml: Add configuration for LicenseCheck UefiCpuPkg/UefiCpuPkg.ci.yaml: Add configuration for LicenseCheck UnitTestFrameworkPkg: Add configuration for LicenseCheck in yaml file .pytool/Plugin/LicenseCheck/LicenseCheck.py | 115 ++++++++++++++++++ .../LicenseCheck/LicenseCheck_plug_in.yaml | 11 ++ .pytool/Plugin/LicenseCheck/Readme.md | 17 +++ ArmVirtPkg/ArmVirtPkg.ci.yaml | 4 + CryptoPkg/CryptoPkg.ci.yaml | 3 + EmulatorPkg/EmulatorPkg.ci.yaml | 4 + FatPkg/FatPkg.ci.yaml | 3 + FmpDevicePkg/FmpDevicePkg.ci.yaml | 3 + MdeModulePkg/MdeModulePkg.ci.yaml | 4 + MdePkg/MdePkg.ci.yaml | 4 + NetworkPkg/NetworkPkg.ci.yaml | 3 + OvmfPkg/OvmfPkg.ci.yaml | 4 + PcAtChipsetPkg/PcAtChipsetPkg.ci.yaml | 3 + SecurityPkg/SecurityPkg.ci.yaml | 3 + ShellPkg/ShellPkg.ci.yaml | 3 + UefiCpuPkg/UefiCpuPkg.ci.yaml | 3 + .../UnitTestFrameworkPkg.ci.yaml | 4 + 17 files changed, 191 insertions(+) create mode 100644 .pytool/Plugin/LicenseCheck/LicenseCheck.py create mode 100644 .pytool/Plugin/LicenseCheck/LicenseCheck_plug_in.yaml create mode 100644 .pytool/Plugin/LicenseCheck/Readme.md -- 2.18.0.windows.1