Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx> writes: > On Fri, Jul 01 2022, Junio C Hamano wrote: > >> * gc/submodule-use-super-prefix (2022-06-30) 8 commits >> - submodule--helper: remove display path helper >> - submodule--helper update: use --super-prefix >> - submodule--helper: remove unused SUPPORT_SUPER_PREFIX flags >> - submodule--helper: use correct display path helper >> - submodule--helper: don't recreate recursive prefix >> - submodule--helper update: use display path helper >> - submodule--helper tests: add missing "display path" coverage >> - Merge branch 'ab/submodule-cleanup' into gc/submodule-use-super-prefix >> (this branch uses ab/submodule-cleanup.) >> >> Another step to rewrite more parts of "git submodule" in C. >> >> Will merge to 'next'? >> source: <20220701021157.88858-1-chooglen@xxxxxxxxxx> > > [...] > >> * ab/submodule-cleanup (2022-06-28) 12 commits >> - git-sh-setup.sh: remove "say" function, change last users >> - git-submodule.sh: use "$quiet", not "$GIT_QUIET" >> - submodule--helper: eliminate internal "--update" option >> - submodule--helper: understand --checkout, --merge and --rebase synonyms >> - submodule--helper: report "submodule" as our name in some "-h" output >> - submodule--helper: rename "absorb-git-dirs" to "absorbgitdirs" >> - submodule update: remove "-v" option >> - submodule--helper: have --require-init imply --init >> - git-submodule.sh: remove unused top-level "--branch" argument >> - git-submodule.sh: make the "$cached" variable a boolean >> - git-submodule.sh: remove unused $prefix variable >> - git-submodule.sh: remove unused sanitize_submodule_env() >> (this branch is used by gc/submodule-use-super-prefix.) >> >> Further preparation to turn git-submodule.sh into a builtin. >> >> Will merge to 'next'. >> source: <cover-v4-00.12-00000000000-20220628T095914Z-avarab@xxxxxxxxx> > > Thanks, with regards to gc/submodule-use-super-prefix that series gets > my enthusiastic (if it helps): > > Reviewed-by: Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx> > > Likewise the ab/submodule-cleanup parent topic has Glen's: > > Reviewed-by: Glen Choo <chooglen@xxxxxxxxxx> > > Per [1] and [2]. I.e. the latest is the v4, but the Reviewed-By was > given to v3 in [1], and [4] notes on v4 that the one remaining concern > was addressed. > > Thanks both, it's great to see us this close to getting rid of another > *.sh built-in. I plan to submit the migration to the builtin/submodule.c > after these two land, possibly after some memory leak fixes to > builtin/submodule--helper.c. > > 1. https://lore.kernel.org/git/kl6lh74aivc6.fsf@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx/ > 2. https://lore.kernel.org/git/kl6l1qv8ivcs.fsf@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx/ Yup, I think this is an accurate summary of both topics. Likewise, I'm really excited to get rid of another *.sh built-in. Thanks!