On 05/02, Stefan Beller wrote: > v2: > * I dropped the RFC patches for reattaching HEAD as that is not the main > concern of this series. > * patch1 is just about dropping cp1 to reusing cp, instead of additionally > "fixing" mem leaks as finish_command does that for us > * reordered the patches, former patch 3 now as patch 2 (avoid auto-discovery > in new working tree manipulator code) > * Regarding former patch2, I wrote: > > Junio wrote: > >> Sounds good (if only because this makes it similar to other > >> codepaths). > >> > >> Is this something whose breakage before the patch is easily > >> demonstrated with a test? > > > I'll try to come up with a test in a reroll. > > done > > Thanks, > Stefan Changes look good. > > > Stefan Beller (3): > submodule_move_head: reuse child_process structure for futher commands > submodule: avoid auto-discovery in new working tree manipulator code > submodule: properly recurse for read-tree and checkout > > submodule.c | 21 +++++++++++---------- > t/lib-submodule-update.sh | 7 +------ > t/t1013-read-tree-submodule.sh | 1 - > t/t2013-checkout-submodule.sh | 1 - > 4 files changed, 12 insertions(+), 18 deletions(-) > > -- > 2.13.0.rc1.19.g083305f9b1 > -- Brandon Williams