diff options
author | Andrew Chow <github@achow101.com> | 2023-02-07 13:46:56 -0500 |
---|---|---|
committer | Andrew Chow <github@achow101.com> | 2023-02-16 12:47:00 -0500 |
commit | 5497c1483097a9b582ef78089a2ce1101b7d722e (patch) | |
tree | 3e0560b203c181a84e999999803c0ca9bcf5a8cf /contrib/verify-commits/verify-commits.py | |
parent | 76923bfa09397568fb8eb72142468a986fc6f790 (diff) | |
download | bitcoin-5497c1483097a9b582ef78089a2ce1101b7d722e.tar.xz |
verify-commits: Use merge-tree in clean merge check
Diffstat (limited to 'contrib/verify-commits/verify-commits.py')
-rwxr-xr-x | contrib/verify-commits/verify-commits.py | 11 |
1 files changed, 3 insertions, 8 deletions
diff --git a/contrib/verify-commits/verify-commits.py b/contrib/verify-commits/verify-commits.py index ee51f66783..5c37fbcbfe 100755 --- a/contrib/verify-commits/verify-commits.py +++ b/contrib/verify-commits/verify-commits.py @@ -95,7 +95,7 @@ def main(): with open(dirname + "/trusted-keys", "r", encoding="utf8") as f: trusted_keys = f.read().splitlines() - # Set commit and branch and set variables + # Set commit and variables current_commit = args.commit if ' ' in current_commit: print("Commit must not contain spaces", file=sys.stderr) @@ -104,7 +104,6 @@ def main(): no_sha1 = True prev_commit = "" initial_commit = current_commit - branch = subprocess.check_output([GIT, 'show', '-s', '--format=%H', initial_commit]).decode('utf8').splitlines()[0] # Iterate through commits while True: @@ -164,15 +163,11 @@ def main(): allow_unclean = current_commit in unclean_merge_allowed if len(parents) == 2 and check_merge and not allow_unclean: current_tree = subprocess.check_output([GIT, 'show', '--format=%T', current_commit]).decode('utf8').splitlines()[0] - subprocess.call([GIT, 'checkout', '--force', '--quiet', parents[0]]) - subprocess.call([GIT, 'merge', '--no-ff', '--quiet', '--no-gpg-sign', parents[1]], stdout=subprocess.DEVNULL) - recreated_tree = subprocess.check_output([GIT, 'show', '--format=format:%T', 'HEAD']).decode('utf8').splitlines()[0] + recreated_tree = subprocess.check_output([GIT, "merge-tree", parents[0], parents[1]]).decode('utf8').splitlines()[0] if current_tree != recreated_tree: print("Merge commit {} is not clean".format(current_commit), file=sys.stderr) - subprocess.call([GIT, 'diff', current_commit]) - subprocess.call([GIT, 'checkout', '--force', '--quiet', branch]) + subprocess.call([GIT, 'diff', recreated_tree, current_tree]) sys.exit(1) - subprocess.call([GIT, 'checkout', '--force', '--quiet', branch]) prev_commit = current_commit current_commit = parents[0] |