I made it one series since the OvmfPkg patches are dependent on the .pytool changes.

Let me know if you see a substantial advantage to split it up.

Regards,
Michael

On 3/22/2022 12:14 PM, Michael D Kinney wrote:
Hi Michael,

Should this be broken up into 2 series?  One for pytool change and one for 
OvmfPkg?

Mike

-----Original Message-----
From: mikub...@linux.microsoft.com <mikub...@linux.microsoft.com>
Sent: Monday, March 21, 2022 6:58 PM
To: devel@edk2.groups.io
Cc: Kinney, Michael D <michael.d.kin...@intel.com>; Gao, Liming 
<gaolim...@byosoft.com.cn>; Sean Brogan
<sean.bro...@microsoft.com>; Bret Barkelew <bret.barke...@microsoft.com>; Ard 
Biesheuvel <ardb+tianoc...@kernel.org>; Yao, Jiewen
<jiewen....@intel.com>; Justen, Jordan L <jordan.l.jus...@intel.com>; Gerd Hoffmann 
<kra...@redhat.com>; Rebecca Cran
<rebe...@bsdio.com>; Peter Grehan <gre...@freebsd.org>; Laszlo Ersek 
<ler...@redhat.com>
Subject: [PATCH v1 0/4] UncrustifyCheck Ignore Support

From: Michael Kubacki <michael.kuba...@microsoft.com>

REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3880
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3875

This patch series:
1. Updates ignore handling in UncrustifyCheck including adding
a new CI plugin configuration option to ignore files in a package.
2. Reverts Uncrustify changes as requested for files in OvmfPkg.
3. Uses the new ignore file option to ignore the files that had
Uncrustify formatting reverted in OvmfPkg.

Cc: Michael D Kinney <michael.d.kin...@intel.com>
Cc: Liming Gao <gaolim...@byosoft.com.cn>
Cc: Sean Brogan <sean.bro...@microsoft.com>
Cc: Bret Barkelew <bret.barke...@microsoft.com>
Cc: Ard Biesheuvel <ardb+tianoc...@kernel.org>
Cc: Jiewen Yao <jiewen....@intel.com>
Cc: Jordan Justen <jordan.l.jus...@intel.com>
Cc: Gerd Hoffmann <kra...@redhat.com>
Cc: Rebecca Cran <rebe...@bsdio.com>
Cc: Peter Grehan <gre...@freebsd.org>
Cc: Laszlo Ersek <ler...@redhat.com>
Signed-off-by: Michael Kubacki <michael.kuba...@microsoft.com>

Michael Kubacki (4):
   .pytool/Plugin/UncrustifyCheck: Update func to return absolute paths
   .pytool/Plugin/UncrustifyCheck: Add ignore file support
   OvmfPkg: Revert Uncrustify formatting in VbeShim.h files
   OvmfPkg: Do not check VbeShim.h formatting with Uncrustify

  .pytool/Plugin/UncrustifyCheck/Readme.md          |    7 +
  .pytool/Plugin/UncrustifyCheck/UncrustifyCheck.py |   30 +-
  OvmfPkg/Bhyve/BhyveRfbDxe/VbeShim.h               | 1811 ++++++++++----------
  OvmfPkg/OvmfPkg.ci.yaml                           |    5 +
  OvmfPkg/QemuVideoDxe/VbeShim.h                    | 1389 ++++++++-------
  5 files changed, 1640 insertions(+), 1602 deletions(-)

--
2.28.0.windows.1








-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#88339): https://edk2.groups.io/g/devel/message/88339
Mute This Topic: https://groups.io/mt/89944179/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to