From: Ross Burton Date: Mon, 30 Oct 2023 11:48:38 +0000 (+0000) Subject: scripts/contrib/patchreview: fix commit identification X-Git-Tag: yocto-5.2~4739 X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=3c04747b681cf6090ba9c77752f6c2f304dbbe17;p=thirdparty%2Fopenembedded%2Fopenembedded-core-contrib.git scripts/contrib/patchreview: fix commit identification git show-ref looks at the _remote_ ref called HEAD, which is fine when it matches the local HEAD but problematic when you're iterating a series of commits. Use rev-parse to resolve the local name to a proper hash. Signed-off-by: Ross Burton Signed-off-by: Richard Purdie --- diff --git a/scripts/contrib/patchreview.py b/scripts/contrib/patchreview.py index f95cadab0c6..bceae06561c 100755 --- a/scripts/contrib/patchreview.py +++ b/scripts/contrib/patchreview.py @@ -257,7 +257,7 @@ if __name__ == "__main__": row = collections.Counter() row["total"] = len(results) row["date"] = subprocess.check_output(["git", "-C", args.directory, "show", "-s", "--pretty=format:%cd", "--date=format:%s"], universal_newlines=True).strip() - row["commit"] = subprocess.check_output(["git", "-C", args.directory, "show-ref", "--hash", "HEAD"], universal_newlines=True).strip() + row["commit"] = subprocess.check_output(["git", "-C", args.directory, "rev-parse", "HEAD"], universal_newlines=True).strip() row['commit_count'] = subprocess.check_output(["git", "-C", args.directory, "rev-list", "--count", "HEAD"], universal_newlines=True).strip() row['recipe_count'] = count_recipes(layers)