[PATCH V2] test-largefiles: partially adapt for Windows

Adrian Buehlmann adrian at cadifra.com
Mon Jul 2 05:02:20 CDT 2012


On 2012-07-02 01:52, Mads Kiilerich wrote:
> Adrian Buehlmann wrote, On 06/30/2012 07:40 PM:
>> # HG changeset patch
>> # User Adrian Buehlmann <adrian at cadifra.com>
>> # Date 1341077463 -7200
>> # Node ID 327cc87b6f59c133172b36f3aa07bf9262c3070d
>> # Parent  e1f2f7134ceda91ff08a203cad48f288c04238b3
>> test-largefiles: partially adapt for Windows
> 
> Thanks, pushed to crew ...
> 
>> +MSYS on Windows doesn't support "rm -Rf ${USERCACHE}/*", so we define a
>> +portable function instead
>> +
>> +  $ wipecache()
>> +  > {
>> +  >   find "${USERCACHE}" -mindepth 1 -print0 | xargs -0 rm -Rf
>> +  > }
> 
> except for this. 'rm -Rf' works fine for me - especially if I quote the 
> expanded variable.

Yeah. I see now that

  rm -Rf "${USERCACHE}"/*

works here as well. Not sure why I didn't try that.

The quoting is however not optional, as for $TESTTMP we have paths like:

 C:\Users\adi\AppData\Local\Temp\hgtests.rjtx9k\test-largefiles.t

> If you still see a problem then please give some details of the problem 
> you see.

No problems left, thanks.


More information about the Mercurial-devel mailing list