git-svn: fix several corner-case and rare bugs with 'commit'

diff --git a/describe.c b/describe.c
index 1e764026b1554e03eacb0915efbf9b475a6e9c84..ff65742615cb6862d3699e4f301d671a7eadc2e2 100644 (file)
--- a/describe.c
+++ b/describe.c
@@ -11,7 +11,6 @@ static const char describe_usage[] =
 static int all = 0;    /* Default to annotated tags only */
 static int tags = 0;   /* But allow any tags if --tags is specified */
 
-#define DEFAULT_ABBREV 8 /* maybe too many */
 static int abbrev = DEFAULT_ABBREV;
 
 static int names = 0, allocs = 0;
@@ -98,7 +97,7 @@ static int compare_names(const void *_a, const void *_b)
        return (a_date > b_date) ? -1 : (a_date == b_date) ? 0 : 1;
 }
 
-static void describe(char *arg)
+static void describe(char *arg, int last_one)
 {
        unsigned char sha1[20];
        struct commit *cmit;
@@ -132,10 +131,12 @@ static void describe(char *arg)
                if (n) {
                        printf("%s-g%s\n", n->path,
                               find_unique_abbrev(cmit->object.sha1, abbrev));
-                       break;
+                       if (!last_one)
+                               clear_commit_marks(cmit, SEEN);
+                       return;
                }
        }
-       clear_commit_marks(cmit, SEEN);
+       die("cannot describe '%s'", sha1_to_hex(cmit->object.sha1));
 }
 
 int main(int argc, char **argv)
@@ -153,7 +154,7 @@ int main(int argc, char **argv)
                        tags = 1;
                else if (!strncmp(arg, "--abbrev=", 9)) {
                        abbrev = strtoul(arg + 9, NULL, 10);
-                       if (abbrev < 4 || 40 <= abbrev)
+                       if (abbrev < MINIMUM_ABBREV || 40 <= abbrev)
                                abbrev = DEFAULT_ABBREV;
                }
                else
@@ -161,10 +162,12 @@ int main(int argc, char **argv)
        }
 
        if (i == argc)
-               describe("HEAD");
+               describe("HEAD", 1);
        else
-               while (i < argc)
-                       describe(argv[i++]);
+               while (i < argc) {
+                       describe(argv[i], (i == argc - 1));
+                       i++;
+               }
 
        return 0;
 }