[edk2-devel] [Patch 0/3] .pytool/Plugin/EccCheck: Remove git reset and optimize

Michael D Kinney posted 3 patches 2 years, 5 months ago
Failed in applying to current master (apply log)
.pytool/Plugin/EccCheck/EccCheck.py | 245 +++++++++++++++++++---------
1 file changed, 172 insertions(+), 73 deletions(-)
[edk2-devel] [Patch 0/3] .pytool/Plugin/EccCheck: Remove git reset and optimize
Posted by Michael D Kinney 2 years, 5 months ago
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2986

* Use temp directory for all operations to prevent any
  changed to git state.
* Remove git reset operation that could corrupt staged
  and local changes.
* Improve performance by removing redundant directory scans
* Improve performance and reduce log file sizes by using
  --output option of git diff to a temp file instead of
  using stdout.

Cc: Sean Brogan <sean.brogan@microsoft.com>
Cc: Bret Barkelew <Bret.Barkelew@microsoft.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Cc: Michael Kubacki <michael.kubacki@microsoft.com>
Signed-off-by: Michael D Kinney <michael.d.kinney@intel.com>

Michael D Kinney (3):
  .pytool/Plugin/EccCheck: Remove RevertCode()
  .pytool/Plugin/EccCheck: Remove temp directory on exception
  .pytool/Plugin/EccCheck: Add performance optimizations

 .pytool/Plugin/EccCheck/EccCheck.py | 245 +++++++++++++++++++---------
 1 file changed, 172 insertions(+), 73 deletions(-)

-- 
2.32.0.windows.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#83924): https://edk2.groups.io/g/devel/message/83924
Mute This Topic: https://groups.io/mt/87253498/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-