Re: FW: [Buildbot-commits] buildbot/buildbot/slave commands.py,1.31,1.32

Yep rmdir /s/q would have worked just fine but would have made a very 
small version dependency - not all versions of windows shells have the 
same behaviour for /s/q in my experiance.  And then you also have to 
worry about people running in shells that are not cmd.exe like cygwin 
or 4nt.

Small issues that most likely would not have been a problem but having 
a python solution makes it even more compatible.

---
Bear
http://code-bear.com

Open Source Applications Foundation (OSAF)
http://www.osafoundation.org

PGP Fingerprint = 9996 719F 973D B11B E111  D770 9331 E822 40B3 CD29


On May 13, 2005, at 7:57 PM, Nick Trout wrote:

> > buildbot/slave/commands.py (rmdirRecursive): added a check
> > to ensure the path passed into rmdirRecursive actually exists.
> > On win32 a non-existant path would generate an exception.
>
> You could just the following for NT/XP:
>
>         rmdir /S /Q dirname
>
> Nick
>
>
>
> -------------------------------------------------------
> This SF.Net email is sponsored by Oracle Space Sweepstakes
> Want to be the first software developer in space?
> Enter now for the Oracle Space Sweepstakes!
> http://ads.osdn.com/?ad_ids93&alloc_id281&op=click
> _______________________________________________
> Buildbot-devel mailing list
> Buildbot-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@xxxxxxxxxxxxxxxx
> https://lists.sourceforge.net/lists/listinfo/buildbot-devel

PGP.sig
Description: This is a digitally signed message part