.gitlab-ci.d/check-patch: tweak output for CI logs
authorAlex Bennée <alex.bennee@linaro.org>
Fri, 13 Nov 2020 17:44:04 +0000 (17:44 +0000)
committerAlex Bennée <alex.bennee@linaro.org>
Mon, 16 Nov 2020 11:08:40 +0000 (11:08 +0000)
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>

.gitlab-ci.d/check-patch.py

index 0ff30ee077409c6e10db5385a32e70d537a11f2b..39e2b403c9eb901079e644234191d1d023c99142 100755 (executable)
@@ -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")