[edk2-devel] [PATCH 0/6] BaseTools: Improve PatchCheck and add GitPostCommitHook

Philippe Mathieu-Daudé posted 6 patches 4 years, 4 months ago
Failed in applying to current master (apply log)
BaseTools/Scripts/GitPostCommitHook.py | 21 ++++++++
BaseTools/Scripts/PatchCheck.py        | 70 ++++++++++++++++++--------
2 files changed, 70 insertions(+), 21 deletions(-)
create mode 100755 BaseTools/Scripts/GitPostCommitHook.py
[edk2-devel] [PATCH 0/6] BaseTools: Improve PatchCheck and add GitPostCommitHook
Posted by Philippe Mathieu-Daudé 4 years, 4 months ago
Hi,

This series add a script that can be use as git-post-commit
hook to run CheckPatch after each commit.

I avoided shell scripts and use Python, so it should be
usable by Visual Studio, but as I don't use it, I haven't
tested there. However it is helpful for my use case, from
a Linux console, because I won't forget to use CheckPatch
again.

In Patch #3 (GitPostCommitHook) I commented a problem with
adding CRLF scripts on Unix, we can not run them directly.

Patch #1 is actually a minor bug fix.

Regards,

Phil.

Cc: Bob Feng <bob.c.feng@intel.com>
Cc: Liming Gao <liming.gao@intel.com>
Cc: Laszlo Ersek <lersek@redhat.com>
Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Cc: Leif Lindholm <leif.lindholm@linaro.org>

Philippe Mathieu-Daude (6):
  BaseTools/PatchCheck: Stop parsing commit message after --- separator
  BaseTools/PatchCheck: Add a --quiet option
  BaseTools/Scripts: Add GitPostCommitHook.py
  BaseTools/PatchCheck: Extract the run_git() function
  BaseTools/PatchCheck: Add printw() function to print warnings
  BaseTools/PatchCheck: Allow to print colored warnings

 BaseTools/Scripts/GitPostCommitHook.py | 21 ++++++++
 BaseTools/Scripts/PatchCheck.py        | 70 ++++++++++++++++++--------
 2 files changed, 70 insertions(+), 21 deletions(-)
 create mode 100755 BaseTools/Scripts/GitPostCommitHook.py

-- 
2.21.0


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#51786): https://edk2.groups.io/g/devel/message/51786
Mute This Topic: https://groups.io/mt/67003925/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-