Re: [PATCHv1] git-p4: workaround p4 removal of client directory

2016-04-29 Thread Luke Diamand
[+Git, Junio] On 29 April 2016 at 15:10, Jacob Smith wrote: > That's the identical patch I'm maintaining internally; it appears to work. Thanks, I think that counts as a "Tested-by" then. Luke > > Sent from my iPhone > >> On Apr 29, 2016, at 8:52 AM, Luke Diamand

Re: [PATCHv1] git-p4: workaround p4 removal of client directory

2016-04-29 Thread Luke Diamand
Adding correct email for Jacob. On 29 April 2016 at 08:40, Luke Diamand wrote: > On some platforms, "p4 sync -f" will remove the workspace > directory after we have just created it; on some it won't. > This causes problems later when git finds itself in an > orphaned directory.

[PATCHv1] git-p4: workaround p4 removal of client directory

2016-04-29 Thread Luke Diamand
This is a proposed fix for the problem reported by Jacob on OSX where "p4 sync -f" removes the client directory. http://www.spinics.net/lists/git/msg274356.html I tried just moving the "cd", but lots of the tests then fail as the "p4 sync" needs to be in the correct directory. Luke Diamand (1):

[PATCHv1] git-p4: workaround p4 removal of client directory

2016-04-29 Thread Luke Diamand
On some platforms, "p4 sync -f" will remove the workspace directory after we have just created it; on some it won't. This causes problems later when git finds itself in an orphaned directory. Workaround this by cd'ing back to the directory after the "p4 sync -f". Signed-off-by: Luke Diamand