Applied in r1094816.
On Mon, Apr 18, 2011 at 18:44, Greg Stein <gst...@gmail.com> wrote: > On Mon, Apr 18, 2011 at 07:04, Alan Wood <alan.w...@clear.net.nz> wrote: >> Hi devs, >> I have just been looking at running the benchmarks and have got to the >> stage where I can >> run it on windows. >> >> This attached patch fixes three issues with the script: >> 1) use of file:// when I'm sure that file:/// is correct from previous >> discussions on this list >> 2) make sure no \ end up is repos url file:// >> 3) at error handler to rmtree to handle the windows read-only files. >> >> I have removed the "which svn" command, not sure how to do this so it is >> only removed on >> windows as I'm not really a python person. >> I hope this is useful. >> I haven't provided a log message as I don't really think it can be applied >> without a bit of >> editing. > > Yes, it will certainly be useful. Thanks! > > I'll fold your changes into the scripts. It is handy because I don't > have a Windows machine for testing. Please go ahead and continue > sending your patches to the list! > > (or if you'd like, we can offer you direct commit access to the > suite1/ directory) > > Cheers, > -g >
