From: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Date: Tue, 08 Nov 2022 18:34:08 GMT
Subject: Re: [PATCH v2 0/9] submodule: tests, cleanup to prepare for built-in
Message-ID: <221108.86v8npqonn.gmgdl@evledraar.gmail.com>
In-Reply-To: <kl6lh6z947ud.fsf@chooglen-macbookpro.roam.corp.google.com>


On Tue, Nov 08 2022, Glen Choo wrote:

> [...]
> Thanks! This addresses all of my comments from v1, and I didn't 
> spot any other issues during a quick 2nd pass.
>
>   Reviewed-by: Glen Choo <chooglen@google.com>

Thanks for the quick review!

One thing I completely forgot in the v2 CL: The 3/9 here is new. Stictly
speaking we could skip it, but I wanted to be able to mark the new test
I added as as leak-free, and fixing the only leak it ran into was
trivial:
https://lore.kernel.org/git/patch-v2-3.9-a3529d7f9e0-20221108T140501Z-avarab@gmail.com/

So rather than having another "small leak fix for submodules" topic
depend on this topic, I think it's OK to just bundle that up with this
one.