2020-09-18 21:29:01 +08:00
|
|
|
#!/usr/bin/env python3
|
|
|
|
#
|
|
|
|
# check-patch.py: run checkpatch.pl across all commits in a branch
|
|
|
|
#
|
|
|
|
# Copyright (C) 2020 Red Hat, Inc.
|
|
|
|
#
|
|
|
|
# SPDX-License-Identifier: GPL-2.0-or-later
|
|
|
|
|
|
|
|
import os
|
|
|
|
import os.path
|
|
|
|
import sys
|
|
|
|
import subprocess
|
|
|
|
|
|
|
|
namespace = "qemu-project"
|
|
|
|
if len(sys.argv) >= 2:
|
|
|
|
namespace = sys.argv[1]
|
|
|
|
|
|
|
|
cwd = os.getcwd()
|
|
|
|
reponame = os.path.basename(cwd)
|
|
|
|
repourl = "https://gitlab.com/%s/%s.git" % (namespace, reponame)
|
|
|
|
|
|
|
|
# GitLab CI environment does not give us any direct info about the
|
|
|
|
# base for the user's branch. We thus need to figure out a common
|
|
|
|
# ancestor between the user's branch and current git master.
|
|
|
|
subprocess.check_call(["git", "remote", "add", "check-patch", repourl])
|
|
|
|
subprocess.check_call(["git", "fetch", "check-patch", "master"],
|
|
|
|
stdout=subprocess.DEVNULL,
|
|
|
|
stderr=subprocess.DEVNULL)
|
|
|
|
|
|
|
|
ancestor = subprocess.check_output(["git", "merge-base",
|
|
|
|
"check-patch/master", "HEAD"],
|
|
|
|
universal_newlines=True)
|
|
|
|
|
|
|
|
ancestor = ancestor.strip()
|
|
|
|
|
2020-10-22 00:31:34 +08:00
|
|
|
log = subprocess.check_output(["git", "log", "--format=%H %s",
|
|
|
|
ancestor + "..."],
|
|
|
|
universal_newlines=True)
|
|
|
|
|
2020-09-18 21:29:01 +08:00
|
|
|
subprocess.check_call(["git", "remote", "rm", "check-patch"])
|
|
|
|
|
2020-10-22 00:31:34 +08:00
|
|
|
if log == "":
|
|
|
|
print("\nNo commits since %s, skipping checks\n" % ancestor)
|
|
|
|
sys.exit(0)
|
|
|
|
|
2020-09-18 21:29:01 +08:00
|
|
|
errors = False
|
|
|
|
|
2020-11-14 01:44:04 +08:00
|
|
|
print("\nChecking all commits since %s...\n" % ancestor, flush=True)
|
2020-09-18 21:29:01 +08:00
|
|
|
|
2020-11-14 01:44:04 +08:00
|
|
|
ret = subprocess.run(["scripts/checkpatch.pl", "--terse", ancestor + "..."])
|
2020-09-18 21:29:01 +08:00
|
|
|
|
|
|
|
if ret.returncode != 0:
|
|
|
|
print(" ❌ FAIL one or more commits failed scripts/checkpatch.pl")
|
|
|
|
sys.exit(1)
|
|
|
|
|
|
|
|
sys.exit(0)
|