From: Junio C Hamano <gitster@pobox.com>
Date: Mon, 3 Mar 2008 21:08:26 +0000 (-0800)
Subject: describe: fix --long output
X-Git-Tag: v1.5.5-rc0~75
X-Git-Url: https://git.tokkee.org/?a=commitdiff_plain;h=870cf7d69872ebc925989b400fb4e7031870db5f;p=git.git

describe: fix --long output

An error while hand-merging broke the new "--long" option.

This should fix it.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
---

diff --git a/builtin-describe.c b/builtin-describe.c
index c549d362b..43be67182 100644
--- a/builtin-describe.c
+++ b/builtin-describe.c
@@ -166,9 +166,11 @@ static void display_name(struct commit_name *n)
 		printf("%s", n->tag->tag);
 	else
 		printf("%s", n->path);
-	if (longformat)
-		printf("-0-g%s",
-		       find_unique_abbrev(n->tag->tagged->sha1, abbrev));
+}
+
+static void show_suffix(int depth, const unsigned char *sha1)
+{
+	printf("-%d-g%s", depth, find_unique_abbrev(sha1, abbrev));
 }
 
 static void describe(const char *arg, int last_one)
@@ -195,7 +197,12 @@ static void describe(const char *arg, int last_one)
 
 	n = cmit->util;
 	if (n) {
+		/*
+		 * Exact match to an existing ref.
+		 */
 		display_name(n);
+		if (longformat)
+			show_suffix(0, n->tag->tagged->sha1);
 		printf("\n");
 		return;
 	}
@@ -281,8 +288,7 @@ static void describe(const char *arg, int last_one)
 
 	display_name(all_matches[0].name);
 	if (abbrev)
-		printf("-%d-g%s", all_matches[0].depth,
-		       find_unique_abbrev(cmit->object.sha1, abbrev));
+		show_suffix(all_matches[0].depth, cmit->object.sha1);
 	printf("\n");
 
 	if (!last_one)