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