threads / patch / 25165

patchcontrib/ciabot: git-describe commit instead of HEAD

Subject: [PATCH] contrib/ciabot: git-describe commit instead of HEAD

## tl;dr

One message between Sep 20, 2010 and Sep 20, 2010. Diffs are folded; open one to read it.

replies: 0people: 1as markdown or json

Sven Eckelmann· Sep 20, 2010, 09:31 UTC · lore

For each commit a shorter version of the name will be generated. This is either the truncated hash or the output of git-describe. The call to git-describe was only made with an empty shell variable instead of an actual commit hash. Thus it only described the current HEAD and not each commit we want to submit to cia.vc.

Signed-off-by: Sven Eckelmann <sven.eckelmann@gmx.de>
---
I send this patch already two weeks ago and got no reaction. This is a
resent in case it was forgotton.
 contrib/ciabot/ciabot.py |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
Show changes to contrib/ciabot/ciabot.py +1 −1
diff --git a/contrib/ciabot/ciabot.py b/contrib/ciabot/ciabot.py
index d0627e0..9775dff 100755
--- a/contrib/ciabot/ciabot.py
+++ b/contrib/ciabot/ciabot.py
@@ -122,7 +122,7 @@ def report(refname, merged):
     branch = os.path.basename(refname)
 
     # Compute a shortnane for the revision
-    rev = do("git describe ${merged} 2>/dev/null") or merged[:12]
+    rev = do("git describe '"+ merged +"' 2>/dev/null") or merged[:12]
 
     # Extract the neta-information for the commit
     rawcommit = do("git cat-file commit " + merged)
-- 
1.7.2.3

← back to recent threads