Re: [PATCH] t9300: wait for background fast-import process to die after killing it

2018-07-20 Thread Elijah Newren
On Thu, Jul 19, 2018 at 3:26 PM, SZEDER Gábor wrote: > The five new tests added to 't9300-fast-import.sh' in 30e215a65c > (fast-import: checkpoint: dump branches/tags/marks even if > object_count==0, 2017-09-28), all with the prefix "V:" in their test > description, run 'git fast-import' in the ba

Re: [PATCH] t9300: wait for background fast-import process to die after killing it

2018-07-19 Thread Eric Rannaud
On Thu, Jul 19, 2018 at 3:26 PM, SZEDER Gábor wrote: > But how do some of these messages end up on the test script's stderr, > why don't we get them from all five tests, and why do they come from > different file/line locations? Thanks for the detailed explanation. Signed-off-by: Eric Rannaud

[PATCH] t9300: wait for background fast-import process to die after killing it

2018-07-19 Thread SZEDER Gábor
The five new tests added to 't9300-fast-import.sh' in 30e215a65c (fast-import: checkpoint: dump branches/tags/marks even if object_count==0, 2017-09-28), all with the prefix "V:" in their test description, run 'git fast-import' in the background and then 'kill' it as part of a 'test_when_finished'