Re: [PATCH v3 00/13] submodule: convert parts of 'update' to C

From: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Date: Mar 3, 2022, 09:58 UTC
Message-ID: <220303.86sfrz5p9t.gmgdl@evledraar.gmail.com>
In-Reply-To: <20220303005727.69270-1-chooglen@google.com>

On Wed, Mar 02 2022, Glen Choo wrote:

Show 5 quoted lines

> Original series: https://lore.kernel.org/git/20220210092833.55360-1-chooglen@google.com
> (I've trimmed the cc list down to the 'most interested' parties)
>
> Thanks for the input, Ævar :) This just fixes up a few small issues in
> the previous version.

And thanks for working on this, there was a reason I was down to nitpicking in v2, this is looking really good & I can't find any issues with this iteration at all.

So I'm happy to give this my:

Reviewed-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>

Junio: It would be great to get this advancing to "next" so that the
subsequent set of migrations to submodule-in-C advancing.

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