• Home
  • Line#
  • Scopes#
  • Navigate#
  • Raw
  • Download
1#!/usr/bin/env python3
2# SPDX-License-Identifier: GPL-2.0-or-later
3#
4# check-patch.py: run checkpatch.pl across all commits in a branch
5#
6# Based on qemu/.gitlab-ci.d/check-patch.py
7#
8# Copyright (C) 2020 Red Hat, Inc.
9# Copyright (C) 2022 Collabora Ltd.
10
11import os
12import os.path
13import sys
14import subprocess
15
16repourl = "https://gitlab.freedesktop.org/%s.git" % os.environ["CI_MERGE_REQUEST_PROJECT_PATH"]
17
18# GitLab CI environment does not give us any direct info about the
19# base for the user's branch. We thus need to figure out a common
20# ancestor between the user's branch and current git master.
21os.environ["GIT_DEPTH"] = "1000"
22subprocess.call(["git", "remote", "remove", "check-patch"], stdout=subprocess.DEVNULL, stderr=subprocess.DEVNULL)
23subprocess.check_call(["git", "remote", "add", "check-patch", repourl])
24subprocess.check_call(["git", "fetch", "check-patch", os.environ["CI_MERGE_REQUEST_TARGET_BRANCH_NAME"]],
25                      stdout=subprocess.DEVNULL,
26                      stderr=subprocess.DEVNULL)
27
28ancestor = subprocess.check_output(["git", "merge-base",
29                                    "check-patch/%s" % os.environ["CI_MERGE_REQUEST_TARGET_BRANCH_NAME"], "HEAD"],
30                                   universal_newlines=True)
31
32ancestor = ancestor.strip()
33
34log = subprocess.check_output(["git", "log", "--format=%H %s",
35                               ancestor + "..."],
36                              universal_newlines=True)
37
38subprocess.check_call(["git", "remote", "rm", "check-patch"])
39
40if log == "":
41    print("\nNo commits since %s, skipping checks\n" % ancestor)
42    sys.exit(0)
43
44errors = False
45
46print("\nChecking all commits since %s...\n" % ancestor, flush=True)
47
48ret = subprocess.run(["scripts/checkpatch.pl",
49                      "--terse",
50                      "--types", os.environ["CHECKPATCH_TYPES"],
51                      "--git", ancestor + "..."])
52
53if ret.returncode != 0:
54    print("    ❌ FAIL one or more commits failed scripts/checkpatch.pl")
55    sys.exit(1)
56
57sys.exit(0)
58