Re: [PATCH v6 0/6] Add Travis CI support

2015-11-23 Thread Luke Diamand
On 19/11/15 08:58, larsxschnei...@gmail.com wrote: From: Lars Schneider diff to v5: * check if PID file still exists on P4D cleanup (thanks Luke) * fix space/tab formatting error * add sleep to timeout loops (thanks Luke) * replace 'date +%s' with platform independent Python function (thanks Er

Re: [PATCH v6 0/6] Add Travis CI support

2015-11-23 Thread Jeff King
On Fri, Nov 20, 2015 at 12:56:04PM +, Luke Diamand wrote: > >>> git-p4: retry kill/cleanup operations in tests with timeout > >>> git-p4: add p4d timeout in tests > >>> git-p4: add trap to kill p4d on test exit > >> > >> These are all fairly gross, and I don't have p4d to test with myself.

Re: [PATCH v6 0/6] Add Travis CI support

2015-11-20 Thread Luke Diamand
On 20 November 2015 at 08:46, Lars Schneider wrote: > > On 19 Nov 2015, at 15:14, Jeff King wrote: > > >> >>> git-p4: retry kill/cleanup operations in tests with timeout >>> git-p4: add p4d timeout in tests >>> git-p4: add trap to kill p4d on test exit >> >> These are all fairly gross, and I d

Re: [PATCH v6 0/6] Add Travis CI support

2015-11-20 Thread Lars Schneider
On 19 Nov 2015, at 15:14, Jeff King wrote: > On Thu, Nov 19, 2015 at 09:58:05AM +0100, larsxschnei...@gmail.com wrote: > >> From: Lars Schneider >> >> diff to v5: >> * check if PID file still exists on P4D cleanup (thanks Luke) >> * fix space/tab formatting error >> * add sleep to timeout loo

Re: [PATCH v6 0/6] Add Travis CI support

2015-11-19 Thread Jeff King
On Thu, Nov 19, 2015 at 09:58:05AM +0100, larsxschnei...@gmail.com wrote: > From: Lars Schneider > > diff to v5: > * check if PID file still exists on P4D cleanup (thanks Luke) > * fix space/tab formatting error > * add sleep to timeout loops (thanks Luke) > * replace 'date +%s' with platform in

[PATCH v6 0/6] Add Travis CI support

2015-11-19 Thread larsxschneider
From: Lars Schneider diff to v5: * check if PID file still exists on P4D cleanup (thanks Luke) * fix space/tab formatting error * add sleep to timeout loops (thanks Luke) * replace 'date +%s' with platform independent Python function (thanks Eric and Luke) With the patches of this series the Tr