diff mbox series

[PULL,9/9] .gitlab-ci.d/check-patch: tweak output for CI logs

Message ID 20201116122417.28346-10-alex.bennee@linaro.org
State Accepted
Commit 7025111a199b97ae806817788bec50f456c47d85
Headers show
Series various fixes (CI, Xen, warnings) | expand

Commit Message

Alex Bennée Nov. 16, 2020, 12:24 p.m. UTC
We don't need running commentary for the CI logs and by keeping it
short we might just see the problem on the first page. While we are at
it flush the previous line so order is maintained between script and
sub process.

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>

Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>

Message-Id: <20201113174404.19608-1-alex.bennee@linaro.org>

-- 
2.20.1
diff mbox series

Patch

diff --git a/.gitlab-ci.d/check-patch.py b/.gitlab-ci.d/check-patch.py
index 0ff30ee077..39e2b403c9 100755
--- a/.gitlab-ci.d/check-patch.py
+++ b/.gitlab-ci.d/check-patch.py
@@ -45,9 +45,9 @@  if log == "":
 
 errors = False
 
-print("\nChecking all commits since %s...\n" % ancestor)
+print("\nChecking all commits since %s...\n" % ancestor, flush=True)
 
-ret = subprocess.run(["scripts/checkpatch.pl", ancestor + "..."])
+ret = subprocess.run(["scripts/checkpatch.pl", "--terse", ancestor + "..."])
 
 if ret.returncode != 0:
     print("    ❌ FAIL one or more commits failed scripts/checkpatch.pl")