From: Ross Burton <[email protected]>

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 <[email protected]>
---
 scripts/contrib/patchreview.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

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)
 
-- 
2.34.1

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#189792): 
https://lists.openembedded.org/g/openembedded-core/message/189792
Mute This Topic: https://lists.openembedded.org/mt/102273104/21656
Group Owner: [email protected]
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to