* Add rmdir, mkdir -p and umask to d8 on Unix....

* Add rmdir, mkdir -p and umask to d8 on Unix.
* Remove the non-working methods from the os object on d8 on Windows
so you can test for their presence with if (os.system).
* Add a test (not run by default since it only works on d8).
* Fix incorrect use of wait that left defunct processes (zombies).

Committed: http://code.google.com/p/v8/source/detail?r=1650

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=56276

[Søren Thygesen Gjesse, 2009-03-31 12:28:01]

http://codereview.chromium.org/56107/diff/1/5
File src/d8-posix.cc (right):

http://codereview.chromium.org/56107/diff/1/5#newcode587
Line 587: if (errno == EEXIST) {
Is this is to handle from paths with '..' e.g. xxx/yyy/../yyy, or is it not
supposed to happen?

http://codereview.chromium.org/56107/diff/1/3
File src/d8-windows.cc (right):

http://codereview.chromium.org/56107/diff/1/3#newcode39
Line 39: os_templ->Set(String::New("setenv"),
FunctionTemplate::New(SetEnvironment));
Why did you leave setenv here? It is not implemented either.

[Erik Corry, 2009-03-31 12:36:30]

http://codereview.chromium.org/56107/diff/1/5
File src/d8-posix.cc (right):

http://codereview.chromium.org/56107/diff/1/5#newcode587
Line 587: if (errno == EEXIST) {
On 2009/03/31 12:28:02, Søren Gjesse wrote:
> Is this is to handle from paths with '..' e.g. xxx/yyy/../yyy, or is it not
> supposed to happen?

Can I just pretend that's what I was thinking about all along?

Test added to ensure that it actually works this way.

http://codereview.chromium.org/56107/diff/1/3
File src/d8-windows.cc (right):

http://codereview.chromium.org/56107/diff/1/3#newcode39
Line 39: os_templ->Set(String::New("setenv"),
FunctionTemplate::New(SetEnvironment));
On 2009/03/31 12:28:02, Søren Gjesse wrote:
> Why did you leave setenv here? It is not implemented either.

Good point.  Removed.