Re: [PATCH v2] t6026: ensure that long-running script really is

2016-11-11 Thread Johannes Sixt
Am 11.11.2016 um 22:09 schrieb Junio C Hamano: Johannes Sixt writes: Am 11.11.2016 um 21:48 schrieb Junio C Hamano: Johannes Sixt writes: Good point. Here is an updated version. Unfortunately, I already took the version before this one and started my integration cycle today. I'll wiggle

Re: [PATCH v2] t6026: ensure that long-running script really is

2016-11-11 Thread Junio C Hamano
Johannes Sixt writes: > Am 11.11.2016 um 21:48 schrieb Junio C Hamano: >> Johannes Sixt writes: >> >>> Good point. Here is an updated version. >> >> Unfortunately, I already took the version before this one and >> started my integration cycle today. I'll wiggle this in; it >> essentially is abo

Re: [PATCH v2] t6026: ensure that long-running script really is

2016-11-11 Thread Johannes Sixt
Am 11.11.2016 um 21:48 schrieb Junio C Hamano: Johannes Sixt writes: Good point. Here is an updated version. Unfortunately, I already took the version before this one and started my integration cycle today. I'll wiggle this in; it essentially is about adding a big comment to explain what is

Re: [PATCH v2] t6026: ensure that long-running script really is

2016-11-11 Thread Junio C Hamano
Johannes Sixt writes: > Good point. Here is an updated version. Unfortunately, I already took the version before this one and started my integration cycle today. I'll wiggle this in; it essentially is about adding a big comment to explain what is going on and then moving the when-finished down

[PATCH v2] t6026: ensure that long-running script really is

2016-11-11 Thread Johannes Sixt
When making sure that background tasks are cleaned up in 5babb5b (t6026-merge-attr: clean up background process at end of test case, 2016-09-07), we considered to let the background task sleep longer, just to be certain that it will still be running when we want to kill it after the test. Sadly, t