Re: [PATCH v3 1/7] refs: plumb repo into ref stores

From: Josh Steadmon <steadmon@google.com>
Date: Oct 7, 2021, 18:33 UTC
Message-ID: <YV89aRzwnP4Jk8x1@google.com>
In-Reply-To: <80673975382af534cafaec3fdc034966d6f41105.1632956589.git.jonathantanmy@google.com>

On 2021.09.29 16:06, Jonathan Tan wrote:

Show 7 quoted lines

> In preparation for the next 2 patches that adds (partial) support for
> arbitrary repositories to ref iterators, plumb a repository into all ref
> stores. There are no changes to program logic.
> 
> (The repository is plumbed into the ref stores instead of directly into
> the ref iterators themselves, so that existing code that operates on ref
> stores do not need to be modified to also handle repositories.)

The second paragraph is a bit confusing, as in patch 2 we do end up adding repository pointers into various iterators. Could you clarify this a bit?

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