From bd3af4391f2df54a33892cba8e366b6eae8cf9f8 Mon Sep 17 00:00:00 2001
From: Ankush Menat <ankush@frappe.io>
Date: Tue, 31 Dec 2024 16:03:57 +0530
Subject: [PATCH] fix: Commit ID parsing (#28975)

This is/was never returning anything.

- Arguments come before revs
- `cd` is kinda unnecessary
---
 frappe/utils/change_log.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/frappe/utils/change_log.py b/frappe/utils/change_log.py
index 10fceee009..f1a6f82dd1 100644
--- a/frappe/utils/change_log.py
+++ b/frappe/utils/change_log.py
@@ -152,7 +152,7 @@ def get_app_last_commit_ref(app):
 	try:
 		with open(os.devnull, "wb") as null_stream:
 			result = subprocess.check_output(
-				f"cd ../apps/{app} && git rev-parse HEAD --short 7",
+				f"git -C ../apps/{app} rev-parse --short=7 HEAD",
 				shell=True,
 				stdin=null_stream,
 				stderr=null_stream,