Re: [PATCH v1] git-p4: add config to retry p4 commands; retry 3 times by default

2016-12-05 Thread Ori Rawlings
Looks good to me, too. -r flag seems to be supported as far back as I can search in the Helix release notes. Ori Rawlings

Re: [PATCH v1] git-p4: add config to retry p4 commands; retry 3 times by default

2016-12-05 Thread Luke Diamand
On 4 December 2016 at 14:03, wrote: > From: Lars Schneider > > P4 commands can fail due to random network issues. P4 users can counter > these issues by using a retry flag supported by all p4 commands [1]. > > Add an integer Git config value `git-p4.retries` to define the number of > retries for

Re: [PATCH v1] git-p4: add config to retry p4 commands; retry 3 times by default

2016-12-05 Thread Lars Schneider
> On 04 Dec 2016, at 15:03, larsxschnei...@gmail.com wrote: > > From: Lars Schneider Hi Junio, if you decide to queue this patch and/or the "git-p4: fix empty file processing for large file system backend GitLFS", please use my signed-off address. I accidentally messed up the author field in

[PATCH v1] git-p4: add config to retry p4 commands; retry 3 times by default

2016-12-04 Thread larsxschneider
From: Lars Schneider P4 commands can fail due to random network issues. P4 users can counter these issues by using a retry flag supported by all p4 commands [1]. Add an integer Git config value `git-p4.retries` to define the number of retries for all p4 invocations. If the config is not defined