Re: git-p4: t9819 failing

2015-09-25 Thread Lars Schneider
Good catch. The case-handling test is actually fine. There was a bug in my implementation! If you do this: git p4 clone //depot/something/... Then git p4 generates a directory “something” and clones into that (similar to Git). Since I set “cloneDirectory” to the current working directory that

Re: git-p4: t9819 failing

2015-09-24 Thread Luke Diamand
On 23 September 2015 at 13:28, Lars Schneider wrote: > >> Here's the last bit of the crash dump from git-p4 I get: >> >> File "/home/ldiamand/git/git/git-p4", line 2580, in streamP4FilesCbSelf >>self.streamP4FilesCb(entry) >> File "/home/ldiamand/git/git/git-p4",

Re: git-p4: t9819 failing

2015-09-24 Thread Luke Diamand
OK, slight correction there - it now doesn't crash getting the disk usage, but I think it still needs to be updated following the other changes to case-handling. Luke On 24 September 2015 at 08:45, Luke Diamand wrote: > On 23 September 2015 at 13:28, Lars Schneider

Re: git-p4: t9819 failing

2015-09-23 Thread Lars Schneider
On 23 Sep 2015, at 11:27, Luke Diamand wrote: > Lars, > > I just noticed that your change "git-p4: honor core.ignorecase when > using P4 client specs" seems to break t9819. > > I suspect that the problem is just that t9819 needs to be updated to > reflect your change - do

Re: git-p4: t9819 failing

2015-09-23 Thread Lars Schneider
On 23 Sep 2015, at 13:11, Luke Diamand wrote: > On 23 September 2015 at 11:27, Lars Schneider > wrote: >> >> On 23 Sep 2015, at 11:27, Luke Diamand wrote: >> >>> Lars, >>> >>> I just noticed that your change "git-p4: honor

Re: git-p4: t9819 failing

2015-09-23 Thread Luke Diamand
On 23 September 2015 at 11:27, Lars Schneider wrote: > > On 23 Sep 2015, at 11:27, Luke Diamand wrote: > >> Lars, >> >> I just noticed that your change "git-p4: honor core.ignorecase when >> using P4 client specs" seems to break t9819. >> >> I suspect

git-p4: t9819 failing

2015-09-23 Thread Luke Diamand
Lars, I just noticed that your change "git-p4: honor core.ignorecase when using P4 client specs" seems to break t9819. I suspect that the problem is just that t9819 needs to be updated to reflect your change - do you have any thoughts on that? Thanks! Luke -- To unsubscribe from this list: send