vezel-dev / cathode

A terminal-centric replacement for the .NET console APIs.
https://docs.vezel.dev/cathode
BSD Zero Clause License
91 stars 7 forks source link

Enable support for killing the entire process tree of a `ChildProcess` #72

Closed alexrp closed 2 years ago

alexrp commented 2 years ago

We currently ignore the value of entireProcessTree since the System.Diagnostics.Process implementation appears to be broken on Windows and we need the Exited event to be fired.

https://github.com/alexrp/system-terminal/blob/91e3a7ad8c80bb9db6fc25cb6dc3810e734d05a1/src/core/Processes/ChildProcess.cs#L169-L185

See: https://github.com/dotnet/runtime/issues/63328