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

Re: [PATCH] commit-reach: parse commits in the given repository

From
Junio C Hamano <gitster@pobox.com>
Date
Sep 16, 2026, 17:48 UTC
Message-ID
<xmqqld91w0ia.fsf@gitster.g>
In-Reply-To
<20260916134632.1424829-1-orestisflo@gmail.com>
Orestis Floros <orestisflo@gmail.com> writes:
Show 11 quoted lines
> ...
> The latter used to be the only symptom. Before bb5da75d61 (commit: use
> commit graph in `lookup_commit_reference_gently()`, 2026-02-16) the
> initial lookup did not record commit-graph positions, so the walk simply
> failed to find the submodule commits in the superproject:
>
>     error: Could not read <commit>
>     Failed to merge submodule sub (commits don't follow merge-base)
>
> Pass the repository into both functions. git-fetch-pack(1) and
> git-upload-pack(1) keep passing `the_repository`.

That's reasonable. It is a very welcome change that aligns with recent libification efforts.

Will queue.
Thanks.
Previous: Kristofer KarlssonNext: Patrick Steinhardt
Message 5 of 6 in “[BUG] "commit graph is likely corrupt" on git rebase”
  1. Florian SchmidtJul 31, 2026
  2. Patrick SteinhardtAug 10, 2026
  3. commit-reach: parse commits in the given repositoryOrestis Floros, Sep 16, 2026
  4. Kristofer KarlssonSep 16, 2026
  5. Junio C HamanoSep 16, 2026
  6. Patrick SteinhardtSep 23, 2026

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.