[Issue 13753] src/std/process.d: _spawnvp error handling is broken
https://issues.dlang.org/show_bug.cgi?id=13753 Iain Buclaw changed: What|Removed |Added Priority|P1 |P3 --
[Issue 13753] src/std/process.d: _spawnvp error handling is broken
https://issues.dlang.org/show_bug.cgi?id=13753 berni44 changed: What|Removed |Added CC||bugzi...@d-ecke.de --- Comment #5 from berni44 --- spawnvp seems only to exist for windows now. But I did not check if this has been the case in 2014 too. Tried to adapt the test for "browse": ``` import std.process; import core.thread; int main() { auto pidBefore = getpid(); try { browse("DOESNOTEXIST"); assert(false); } catch(Exception e) { auto pidAfterwards = getpid(); assert(pidBefore == pidAfterwards); // make sure we are still in the same process } return 0; } ``` This produces an Assertion failure (POSIX). And the process seems still to be running after the main process has stopped. --
[Issue 13753] src/std/process.d: _spawnvp error handling is broken
https://issues.dlang.org/show_bug.cgi?id=13753 --- Comment #4 from Danny Milosavljevic --- Also, both the OSX and the Posix version of browse in the same file are broken in the same way... --
[Issue 13753] src/std/process.d: _spawnvp error handling is broken
https://issues.dlang.org/show_bug.cgi?id=13753 Danny Milosavljevic changed: What|Removed |Added Summary|src/std/process.d: _spawnvp |src/std/process.d: _spawnvp |is broken |error handling is broken --