git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: Bug in 'git describe' if I have two tags on the same commit.

From
Junio C Hamano <gitster@pobox.com>
Date
Feb 13, 2017, 20:35 UTC
Message-ID
<xmqqo9y5de1k.fsf@gitster.mtv.corp.google.com>
In-Reply-To
<20170213154407.GA31568@alpha.ikke.info>
Kevin Daudt <me@ikke.info> writes:
Show 15 quoted lines
> On Sun, Feb 12, 2017 at 01:15:22PM +0100, Istvan Pato wrote:
>
>> (master) [1] % git show-ref --tag
>> 76c634390... refs/tags/1.0.0
>> b77c7cd17... refs/tags/1.1.0
>> b77c7cd17... refs/tags/1.2.0
>> 
>> (master) % git describe --tags --always
>> 1.1.0-1-ge9e9ced
>> 
>> ### Expected: 1.2.0
>> ...
>
> Are these lightweight tags? Only annotated tags have a date associated
> to them, which is where the rel-notes refers to. 

Good eyes. The fact that the two points at the same object means that even if they were both annotated tags, they have the same tagger dates.

If the code that compares the candidates and picks better tag to describe the object with knows the refnames of these "tags", I'd imagine that we could use the versioncmp() logic as the final tie breaker, but I do not offhand remember if the original refname we took the tag (or commit) from is propagated that deep down the callchain. It probably does not, so some code refactoring may be needed if somebody wants to go in that direction.

Previous: Kevin DaudtNext: Istvan Pato
Message 3 of 4 in “Bug in 'git describe' if I have two tags on the same commit.”
  1. Istvan PatoFeb 12, 2017
  2. Kevin DaudtFeb 13, 2017
  3. Junio C HamanoFeb 13, 2017
  4. Istvan PatoFeb 14, 2017

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.