Re: [PATCH v3 0/9] submodule: convert the rest of 'update' to C
- From
Emily Shaffer <emilyshaffer@google.com>
- Date
- Oct 14, 2021, 20:46 UTC
- Message-ID
- <YWiXL+plA7GHfuVv@google.com>
- In-Reply-To
- <xmqqczo8eahl.fsf@gitster.g>
On Wed, Oct 13, 2021 at 05:05:58PM -0700, Junio C Hamano wrote:
Show 19 quoted lines
> > Atharva Raykar <raykar.ath@gmail.com> writes: > > > I have attempted to make a version of this series that is based on that topic [2], > > and added the superproject gitdir caching code in C [3]. It passes the tests, > > but I am not too confident about its correctness. I hope that branch can be > > helpful in some way. > > > > [1] https://lore.kernel.org/git/20210819200953.2105230-1-emilyshaffer@google.com/ > > [2] https://github.com/tfidfwastaken/git/commits/submodule-update-on-es-superproject-aware > > (fetch-it-via: git fetch https://github.com/tfidfwastaken/git submodule-update-on-es-superproject-aware) > > [3] https://github.com/tfidfwastaken/git/blob/a74aaf2540c536970f2541d3042c825f82a69770/builtin/submodule--helper.c#L2922-L2930 > > The "C rewrite" of the code [3] that unconditionally sets of the > submodule.superprojectgitdir varible seems straightforward enough. > > Emily, how solid do you think your "superproject aware submodule" > topic already is? Would it be stable enough to build other things > on top, or is it a bit too premature?
As of the version I sent today (https://lore.kernel.org/git/20211014203416.2802639-1-emilyshaffer%40google.com) I think it is stable enough to build on top of. There was general consensus on the semantics of submodule.superprojectgitdir as it's sent in v4.
- Emily