Re: [PATCH v2] git-p4: Ask "p4" to interpret View setting

2013-08-29 Thread Pete Wyckoff
ksaitoh...@gmail.com wrote on Tue, 27 Aug 2013 11:43 +0900: > > Do you have an updated patch? Want to take some time to clean up and > > resubmit the entire series? The batching should be incorporated with > > the last 2/2 that I sent out. > > I don't have other update. > I'm satisfied because a

Re: [PATCH v2] git-p4: Ask "p4" to interpret View setting

2013-08-26 Thread kazuki saitoh
> Do you have an updated patch? Want to take some time to clean up and > resubmit the entire series? The batching should be incorporated with > the last 2/2 that I sent out. I don't have other update. I'm satisfied because able to want to do and it became better than my original modification tha

Re: [PATCH v2] git-p4: Ask "p4" to interpret View setting

2013-08-24 Thread Pete Wyckoff
p...@padd.com wrote on Thu, 15 Aug 2013 21:24 -0400: > ksaitoh...@gmail.com wrote on Wed, 14 Aug 2013 09:59 +0900: > > > My only concern is in the commit message, about performance. A > > > change that has lots of files in it will cause many roundtrips to > > > p4d to do "p4 where" on each. When

Re: [PATCH v2] git-p4: Ask "p4" to interpret View setting

2013-08-15 Thread Pete Wyckoff
ksaitoh...@gmail.com wrote on Wed, 14 Aug 2013 09:59 +0900: > > My only concern is in the commit message, about performance. A > > change that has lots of files in it will cause many roundtrips to > > p4d to do "p4 where" on each. When the files don't have much > > edited content, this new approa

Re: [PATCH v2] git-p4: Ask "p4" to interpret View setting

2013-08-10 Thread Pete Wyckoff
ksaitoh...@gmail.com wrote on Tue, 06 Aug 2013 15:45 +0900: > In Perforce, View setting of p4 client can describe > -//depot/project/files/*.xls //client/project/files/*.xls > to exclude Excel files. > But "git p4 --use-client-spec" cannot support '*'. > > In git-p4.py, "map_in_client" method an

[PATCH v2] git-p4: Ask "p4" to interpret View setting

2013-08-05 Thread kazuki saitoh
In Perforce, View setting of p4 client can describe -//depot/project/files/*.xls //client/project/files/*.xls to exclude Excel files. But "git p4 --use-client-spec" cannot support '*'. In git-p4.py, "map_in_client" method analyzes View setting and return client file path. So I modify the method