[Issue 16615] std.process is missing functionality for child processes

[d-bugmail--- via Digitalmars-d-bugs]

https://issues.dlang.org/show_bug.cgi?id=16615

--- Comment #7 from github-bugzi...@puremagic.com ---
Commits pushed to dmd-cxx at https://github.com/dlang/phobos

https://github.com/dlang/phobos/commit/71ca4c5b1428e6b4a9b2d89586cd8851c48619a6
Partial Fix For Issue 16615 - convert os pid to std.process Pid

https://github.com/dlang/phobos/commit/15a5a4898c67d0ecb7c7d2a5d9e744eeaf188ca1
Merge pull request #5086 from edi33416/process_enhancement

https://github.com/dlang/phobos/commit/07602da6334fa250032d302c777b75d17b0e4379
Revert "Partial Fix Issue 16615 - convert os pid to std.process Pid"

https://github.com/dlang/phobos/commit/9bfc82130c0e4af4d1dc95bb261570c6e4f6f5d8
Merge pull request #5456 from dlang/revert-5086-process_enhancement

--

[Issue 16615] std.process is missing functionality for child processes

[via Digitalmars-d-bugs]

https://issues.dlang.org/show_bug.cgi?id=16615

--- Comment #6 from github-bugzi...@puremagic.com ---
Commits pushed to stable at https://github.com/dlang/phobos

https://github.com/dlang/phobos/commit/71ca4c5b1428e6b4a9b2d89586cd8851c48619a6
Partial Fix For Issue 16615 - convert os pid to std.process Pid

https://github.com/dlang/phobos/commit/15a5a4898c67d0ecb7c7d2a5d9e744eeaf188ca1
Merge pull request #5086 from edi33416/process_enhancement

https://github.com/dlang/phobos/commit/07602da6334fa250032d302c777b75d17b0e4379
Revert "Partial Fix Issue 16615 - convert os pid to std.process Pid"

https://github.com/dlang/phobos/commit/9bfc82130c0e4af4d1dc95bb261570c6e4f6f5d8
Merge pull request #5456 from dlang/revert-5086-process_enhancement

--

[Issue 16615] std.process is missing functionality for child processes

[via Digitalmars-d-bugs]

https://issues.dlang.org/show_bug.cgi?id=16615

github-bugzi...@puremagic.com changed:

   What|Removed |Added

 Resolution|WONTFIX |FIXED

--

[Issue 16615] std.process is missing functionality for child processes

[via Digitalmars-d-bugs]

https://issues.dlang.org/show_bug.cgi?id=16615

Vladimir Panteleev  changed:

   What|Removed |Added

 CC||dhase...@gmail.com

--- Comment #5 from Vladimir Panteleev  ---
*** Issue 17479 has been marked as a duplicate of this issue. ***

--

[Issue 16615] std.process is missing functionality for child processes

[via Digitalmars-d-bugs]

https://issues.dlang.org/show_bug.cgi?id=16615

Vladimir Panteleev  changed:

   What|Removed |Added

 CC||thecybersha...@gmail.com
 Resolution|FIXED   |WONTFIX

--- Comment #4 from Vladimir Panteleev  ---
Resolving as WONTFIX as per
https://github.com/dlang/phobos/pull/5086#issuecomment-307461557 and
https://github.com/dlang/phobos/pull/5086#issuecomment-307426691 (quoting):

Although it might not be obvious, killing a process is inherently a very
platform specific operation. There is and cannot be an obviously correct way to
do it on both platforms (Windows and POSIX), because the two platforms manage
processes in different way. Windows does not have signals; POSIX does not allow
specifying exit codes; and reparenting is done differently on both. 

Furthermore, the example use case presented in the issue (killing a process
tree) is not possible to correctly implement with just kill - you must suspend
the entire process group, and only then kill them, and I'm not sure that even
that is sufficient. I would advise against attempting to do it in general, and
instead run the process group in a container and kill it wholesale.

--

[Issue 16615] std.process is missing functionality for child processes

[via Digitalmars-d-bugs]

https://issues.dlang.org/show_bug.cgi?id=16615

--- Comment #3 from github-bugzi...@puremagic.com ---
Commits pushed to master at https://github.com/dlang/phobos

https://github.com/dlang/phobos/commit/07602da6334fa250032d302c777b75d17b0e4379
Revert "Partial Fix Issue 16615 - convert os pid to std.process Pid"

https://github.com/dlang/phobos/commit/9bfc82130c0e4af4d1dc95bb261570c6e4f6f5d8
Merge pull request #5456 from dlang/revert-5086-process_enhancement

Revert "Partial Fix Issue 16615 - convert os pid to std.process Pid"
merged-on-behalf-of: Sebastian Wilzbach 

--

[Issue 16615] std.process is missing functionality for child processes

[via Digitalmars-d-bugs]

https://issues.dlang.org/show_bug.cgi?id=16615

--- Comment #2 from github-bugzi...@puremagic.com ---
Commit pushed to revert-5086-process_enhancement at
https://github.com/dlang/phobos

https://github.com/dlang/phobos/commit/07602da6334fa250032d302c777b75d17b0e4379
Revert "Partial Fix Issue 16615 - convert os pid to std.process Pid"

--

[Issue 16615] std.process is missing functionality for child processes

[via Digitalmars-d-bugs]

https://issues.dlang.org/show_bug.cgi?id=16615

Vladimir Panteleev  changed:

   What|Removed |Added

   See Also||https://issues.dlang.org/sh
   ||ow_bug.cgi?id=17479

--

[Issue 16615] std.process is missing functionality for child processes

[via Digitalmars-d-bugs]

https://issues.dlang.org/show_bug.cgi?id=16615

--- Comment #1 from github-bugzi...@puremagic.com ---
Commits pushed to master at https://github.com/dlang/phobos

https://github.com/dlang/phobos/commit/71ca4c5b1428e6b4a9b2d89586cd8851c48619a6
Partial Fix For Issue 16615 - convert os pid to std.process Pid

https://github.com/dlang/phobos/commit/15a5a4898c67d0ecb7c7d2a5d9e744eeaf188ca1
Merge pull request #5086 from edi33416/process_enhancement

Partial Fix Issue 16615 - convert os pid to std.process Pid
merged-on-behalf-of: Andrei Alexandrescu 

--

[Issue 16615] std.process is missing functionality for child processes

[via Digitalmars-d-bugs]

https://issues.dlang.org/show_bug.cgi?id=16615

github-bugzi...@puremagic.com changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED

--

[Issue 16615] std.process is missing functionality for child processes

[via Digitalmars-d-bugs]

https://issues.dlang.org/show_bug.cgi?id=16615

Andrei Alexandrescu  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED

--

[Issue 16615] std.process is missing functionality for child processes

[via Digitalmars-d-bugs]

https://issues.dlang.org/show_bug.cgi?id=16615

Andrei Alexandrescu  changed:

   What|Removed |Added

 CC||and...@erdani.com
   Assignee|nob...@puremagic.com|edi33...@gmail.com

--