public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* [PATCH v4 00/17] Add a pipline to check Ecc issues for edk2 on open ci
@ 2020-06-15  8:48 Zhang, Shenglei
  2020-06-15  8:48 ` [PATCH v4 01/17] BaseTools/Scripts: Add EccCheck.py Zhang, Shenglei
                   ` (16 more replies)
  0 siblings, 17 replies; 23+ messages in thread
From: Zhang, Shenglei @ 2020-06-15  8:48 UTC (permalink / raw)
  To: devel; +Cc: Bob Feng, Bret Barkelew, Michael D Kinney, Liming Gao,
	Sean Brogan

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2606
As planed we will enable Ecc check for edk2 on open ci. And they are
ready now, but these are V3 series. So I expect that contributors in
edk2 community can try using this script when reviewing. And I appreciate
receiving feedback and comments if someone find errors or false positive
issues.

I created a pipline of EccCheck for my forked edk2.
https://dev.azure.com/shengleizhang/shengleizhang/_build?definitionId=10&_a=summary

The patch series are big, so the commits are also pushed into my forked tree.
https://github.com/shenglei10/edk2/commits/ECC

Patches
1/17: EccCheck.py is a tool to report Ecc issues for commits. It can be run
     on azure servers for open ci, or locally. Its usage is like
     PatchCheck.py.
2/17: It's a lib necessary for py3 to run Ecc on azure servers. For local
     use, we need to type command
     "py -3 -m pip install antlr4-python3-runtime==4.7.1" first.
3/17: Windows-EccCheck.yml is a yaml file to configure the newly added
     pipline. Azure uses this to create a pipline.
4/17~17/17: We consider some cases that will report out Ecc issues but they won't
     be fixed, like submodule and industry standard related things. So we
     add two configuration fields "Exception" and "IgnoreFiles" for people
     to use. These patches add configuration in yaml files for Ecc check.

Cc: Bob Feng <bob.c.feng@intel.com>
Cc: Bret Barkelew <Bret.Barkelew@microsoft.com>
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Liming Gao <liming.gao@intel.com>
Cc: Sean Brogan <sean.brogan@microsoft.com>

v2: Update 1/17, fix the bug that the script can't hanlde multiple commits.

v3: Update 1/17, set the only workalbe workspace is edk2 root directory.
    Update 2/17, designate the version of antlr4 is 4.7.1.
    Add 4/17~17/17.

v4. Update 1/17, remove the function EdksetupRebuild(), instead add
    function SetupEnvironment(). Update variables' format and type hints
    to pass flake8 and mypy.

Shenglei Zhang (17):
  BaseTools/Scripts: Add EccCheck.py
  pip-requirements.txt: Add Ecc required lib
  .azurepiplines: Add a pipline to check ECC issues for commits
  MdeModulePkg/MdeModulePkg.ci.yaml: Add configuration for Ecc check
  ArmVirtPkg/ArmVirtPkg.ci.yaml: Add configuration for Ecc check
  CryptoPkg/CryptoPkg.ci.yaml: Add configuration for Ecc check
  EmulatorPkg/EmulatorPkg.ci.yaml: Add configuration for Ecc check
  FatPkg/FatPkg.ci.yaml: Add configuration for Ecc check
  FmpDevicePkg/FmpDevicePkg.ci.yaml: Add configuration for Ecc check
  MdePkg/MdePkg.ci.yaml: Add configuration for Ecc check
  NetworkPkg/NetworkPkg.ci.yaml: Add configuration for Ecc check
  OvmfPkg/OvmfPkg.ci.yaml: Add configuration for Ecc check
  PcAtChipsetPkg/PcAtChipsetPkg.ci.yaml: Add configuration for Ecc check
  SecurityPkg/SecurityPkg.ci.yaml: Add configuration for Ecc check
  ShellPkg/ShellPkg.ci.yaml: Add configuration for Ecc check
  UefiCpuPkg/UefiCpuPkg.ci.yaml: Add configuration for Ecc check
  UnitTestFrameworkPkg: Add configuration for Ecc check in yaml file

 .azurepipelines/Windows-EccCheck.yml          |  38 ++
 ArmVirtPkg/ArmVirtPkg.ci.yaml                 |  10 +
 BaseTools/Scripts/EccCheck.py                 | 419 ++++++++++++++++++
 CryptoPkg/CryptoPkg.ci.yaml                   |  10 +
 EmulatorPkg/EmulatorPkg.ci.yaml               |  10 +
 FatPkg/FatPkg.ci.yaml                         |  10 +
 FmpDevicePkg/FmpDevicePkg.ci.yaml             |  10 +
 MdeModulePkg/MdeModulePkg.ci.yaml             |  10 +
 MdePkg/MdePkg.ci.yaml                         |  10 +
 NetworkPkg/NetworkPkg.ci.yaml                 |  10 +
 OvmfPkg/OvmfPkg.ci.yaml                       |  10 +
 PcAtChipsetPkg/PcAtChipsetPkg.ci.yaml         |  10 +
 SecurityPkg/SecurityPkg.ci.yaml               |  10 +
 ShellPkg/ShellPkg.ci.yaml                     |  10 +
 UefiCpuPkg/UefiCpuPkg.ci.yaml                 |  10 +
 .../UnitTestFrameworkPkg.ci.yaml              |  10 +
 pip-requirements.txt                          |   1 +
 17 files changed, 598 insertions(+)
 create mode 100644 .azurepipelines/Windows-EccCheck.yml
 create mode 100644 BaseTools/Scripts/EccCheck.py

-- 
2.18.0.windows.1


^ permalink raw reply	[flat|nested] 23+ messages in thread

end of thread, other threads:[~2020-06-29  1:39 UTC | newest]

Thread overview: 23+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-06-15  8:48 [PATCH v4 00/17] Add a pipline to check Ecc issues for edk2 on open ci Zhang, Shenglei
2020-06-15  8:48 ` [PATCH v4 01/17] BaseTools/Scripts: Add EccCheck.py Zhang, Shenglei
2020-06-15  8:48 ` [PATCH v4 02/17] pip-requirements.txt: Add Ecc required lib Zhang, Shenglei
2020-06-15  8:48 ` [PATCH v4 03/17] .azurepiplines: Add a pipline to check ECC issues for commits Zhang, Shenglei
2020-06-15  8:48 ` [PATCH v4 04/17] MdeModulePkg/MdeModulePkg.ci.yaml: Add configuration for Ecc check Zhang, Shenglei
2020-06-15  8:48 ` [PATCH v4 05/17] ArmVirtPkg/ArmVirtPkg.ci.yaml: " Zhang, Shenglei
2020-06-15  8:48 ` [PATCH v4 06/17] CryptoPkg/CryptoPkg.ci.yaml: " Zhang, Shenglei
2020-06-29  1:35   ` [edk2-devel] " Guomin Jiang
2020-06-15  8:48 ` [PATCH v4 07/17] EmulatorPkg/EmulatorPkg.ci.yaml: " Zhang, Shenglei
2020-06-15  8:48 ` [PATCH v4 08/17] FatPkg/FatPkg.ci.yaml: " Zhang, Shenglei
2020-06-29  1:38   ` [edk2-devel] " Guomin Jiang
2020-06-15  8:48 ` [PATCH v4 09/17] FmpDevicePkg/FmpDevicePkg.ci.yaml: " Zhang, Shenglei
2020-06-29  1:36   ` [edk2-devel] " Guomin Jiang
2020-06-15  8:48 ` [PATCH v4 10/17] MdePkg/MdePkg.ci.yaml: " Zhang, Shenglei
2020-06-15  8:48 ` [PATCH v4 11/17] NetworkPkg/NetworkPkg.ci.yaml: " Zhang, Shenglei
2020-06-15  8:48 ` [PATCH v4 12/17] OvmfPkg/OvmfPkg.ci.yaml: " Zhang, Shenglei
2020-06-15  8:48 ` [PATCH v4 13/17] PcAtChipsetPkg/PcAtChipsetPkg.ci.yaml: " Zhang, Shenglei
2020-06-29  1:39   ` [edk2-devel] " Guomin Jiang
2020-06-15  8:48 ` [PATCH v4 14/17] SecurityPkg/SecurityPkg.ci.yaml: " Zhang, Shenglei
2020-06-15  8:48 ` [PATCH v4 15/17] ShellPkg/ShellPkg.ci.yaml: " Zhang, Shenglei
2020-06-24  6:23   ` [edk2-devel] " Gao, Zhichao
2020-06-15  8:48 ` [PATCH v4 16/17] UefiCpuPkg/UefiCpuPkg.ci.yaml: " Zhang, Shenglei
2020-06-15  8:48 ` [PATCH v4 17/17] UnitTestFrameworkPkg: Add configuration for Ecc check in yaml file Zhang, Shenglei

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox