On Wed, Nov 10, 2010 at 11:38:58AM -0800, Junio C Hamano <gitster@xxxxxxxxx> wrote: > Just a guess. Does this have to do with 80dbae0 (Chose better tag names > in git-describe after merges., 2007-01-10)? Hmm, not sure. I tried reverting that commit, but that resulted in conflicts - git checkout 80dbae0^ produces a git binary that can't read the index version we have today. :) But I think the approach to follow the first parent only would solve the problem outlined in the commit message of 80dbae0 as well: I think it's pretty rare to tag a commit in a feature branch, then merge it. I mean the trivial proof of concenpt below finds the right tag here: $ ~/git/git/git describe 1.4pre1-572-g42b497b Even if it does not count all the commits since that tag: $ git rev-list 1.4pre1..|wc -l 883 Thanks. diff --git a/builtin/describe.c b/builtin/describe.c index 43caff2..f32eaf4 100644 --- a/builtin/describe.c +++ b/builtin/describe.c @@ -303,6 +303,7 @@ static void describe(const char *arg, int last_one) insert_by_date(p, &list); p->object.flags |= c->object.flags; parents = parents->next; + break; } }
Attachment:
pgpmVdc65oJ31.pgp
Description: PGP signature