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

[POSSIBLE REGRESSION] Spurious revs after patch "revision.c: --full-history fix"

From
MCMarco Costalba <mcostalba@gmail.com>
Date
Jul 2, 2006, 12:19 UTC
Message-ID
<e5bfff550607020519k6007f41bne34d10c0e919f3c8@mail.gmail.com>
After patch "revision.c: --full-history fix" (6631c73)
I have this in git tree.

I have _manually_ ( --abbrev=nr does not apply in this case) truncated to 7 chars the SHA to be more readable)

$ git-rev-list --topo-order --parents --remove-empty HEAD -- builtin-add.c 021b6e4 93872e0 93872e0 3c6a370 3c6a370 e8f990b e8f990b f259339 f259339 0d78153 0d78153 b4189aa 283c8ee $ git-rev-list --topo-order --remove-empty HEAD -- builtin-add.c 021b6e4 93872e0 3c6a370 e8f990b f259339 0d78153 $

What it seems is that with --parents option two more spurious revs are printed out. This revs have nothing to do with builtin-add.c, the file is newer then both of them.

I have tested with different files and always we have spurious revisions in case of files that were added after initial commit, and always the erroneous revisions are older then file creation one.

This is a possible regression that breaks things (I've found it thanks to an assert in qgit).

   Marco
Next: Linus Torvalds
Message 1 of 4 in “[POSSIBLE REGRESSION] Spurious revs after patch "revision.c: --full-history fix"”
  1. Marco CostalbaJul 2, 2006
  2. Linus TorvaldsJul 2, 2006
  3. Junio C HamanoJul 2, 2006
  4. Marco CostalbaJul 2, 2006

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.