Re: [PATCHv2 0/3] Some submodule bugfixes
On 05/02, Stefan Beller wrote:
Show 21 quoted lines
> 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
Show 16 quoted lines
>
>
> 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