Re: duplicate objects in packfile

From: Junio C Hamano <gitster@pobox.com>
Date: Aug 14, 2013, 18:29 UTC
Message-ID: <7v4nast9bv.fsf@alter.siamese.dyndns.org>
In-Reply-To: <20130814181718.GA7911@sigill.intra.peff.net>

Jeff King <peff@peff.net> writes:

Show 9 quoted lines

> lookup does not.  It hits an assert() that can only be triggered in the
> face of duplicate objects. For example:
>
>   $ git cat-file -t 4ea4acbcb0930ac42acc87a0d203864dec1a9697
>   commit
>
>   $ GIT_USE_LOOKUP=1 git cat-file -t 4ea4acbcb0930ac42acc87a0d203864dec1a9697
>   git: sha1-lookup.c:222: sha1_entry_pos: Assertion `lov < hiv' failed.
>   Aborted

Older versions of JGit used to put duplicate objects in a pack, and IIRC Shawn declared that a bug in the packer and fixed it, so from that point of view, I think rejecting is probably the right thing, even though I think warning and continuing is also acceptable and indeed may be better.

$ 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.