Thread overview
[Issue 13753] src/std/process.d: _spawnvp error handling is broken
[Issue 13753] src/std/process.d: _spawnvp is broken
Dec 17, 2022
Iain Buclaw
November 19, 2014
https://issues.dlang.org/show_bug.cgi?id=13753

--- Comment #1 from Danny Milosavljevic <danny.milo@gmail.com> ---
Created attachment 1454
  --> https://issues.dlang.org/attachment.cgi?id=1454&action=edit
Tests whether spawnvp magically forks processes for the caller

Tests whether spawnvp magically switches processes. Try with version(Posix) implementation of spawnvp.

--
November 19, 2014
https://issues.dlang.org/show_bug.cgi?id=13753

--- Comment #2 from Danny Milosavljevic <danny.milo@gmail.com> ---
Created attachment 1455
  --> https://issues.dlang.org/attachment.cgi?id=1455&action=edit
Patch to make _spawnvp less bad

checks waitpid() return value, does not magically put the caller into a new process. Does not properly handle EINTR.

--
November 19, 2014
https://issues.dlang.org/show_bug.cgi?id=13753

--- Comment #3 from Danny Milosavljevic <danny.milo@gmail.com> ---
See http://www.gnu.org/software/libc/manual/html_node/Interrupted-Primitives.html

--
November 20, 2014
https://issues.dlang.org/show_bug.cgi?id=13753

Danny Milosavljevic <danny.milo@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|src/std/process.d: _spawnvp |src/std/process.d: _spawnvp
                   |is broken                   |error handling is broken

--
November 25, 2014
https://issues.dlang.org/show_bug.cgi?id=13753

--- Comment #4 from Danny Milosavljevic <danny.milo@gmail.com> ---
Also, both the OSX and the Posix version of browse in the same file are broken in the same way...

--
December 17, 2022
https://issues.dlang.org/show_bug.cgi?id=13753

Iain Buclaw <ibuclaw@gdcproject.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P1                          |P3

--