Re: [PATCH 1/2] t5526: check for name/path collision in submodule fetch

2017-10-23 Thread Stefan Beller
>> + git add submodule && > > A small note even though this is not meant for inclusion: This would > break when I start working on teaching 'git add' to set default values > in .gitmodules when available. Yes. I should have used something like: git update-index --add --cacheinfo 1

Re: [PATCH 1/2] t5526: check for name/path collision in submodule fetch

2017-10-23 Thread Heiko Voigt
On Thu, Oct 19, 2017 at 11:11:08AM -0700, Stefan Beller wrote: > Signed-off-by: Stefan Beller > --- > > This is just to test the corner case we're discussing. > Applies on top of origin/hv/fetch-moved-submodules-on-demand. > > > t/t5526-fetch-submodules.sh | 42

[PATCH 1/2] t5526: check for name/path collision in submodule fetch

2017-10-19 Thread Stefan Beller
Signed-off-by: Stefan Beller --- This is just to test the corner case we're discussing. Applies on top of origin/hv/fetch-moved-submodules-on-demand. t/t5526-fetch-submodules.sh | 42 ++ 1 file changed, 42 insertions(+) diff --git a/t/t5526-fetch-submod