Re: [PATCH 19/49] repack: remove 'existing_packs' API from the builtin

From: Patrick Steinhardt <ps@pks.im>
Date: Sep 29, 2025, 23:21 UTC
Message-ID: <aNsUhEVUmzmZ5gp3@pks.im>
In-Reply-To: <ae267fcc820dc19497f9f897bc5915b2040739e5.1759097191.git.me@ttaylorr.com>

On Sun, Sep 28, 2025 at 06:08:23PM -0400, Taylor Blau wrote:

Show 8 quoted lines

> The repack builtin defines an API for keeping track of which packs
> were found in the repository at the beginning of the repack operation.
> This is used to classify what state a pack was in (kept, non-kept, or
> cruft), and is also used to mark which packs to delete (or keep) at the
> end of a repack operation.
> 
> Now that the prerequisite refactoring is complete, this API is isolated
> enough that it can be moved out to repack.ch and removed from the

Tiniest nit, sorry: I found the spelling of "repack.ch" a bit confusing at first, as it looked like a typo. Maybe say "repack.{c,h}" or "repack.[ch]"?

Patrick

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