Request for feedback on new .NET Process APIs #318
Replies: 1 comment
-
|
Hi Adam. These changes are a welcome addition to .NET as it has well-recognized issues when dealing with child processes. I think any new APIs that simplify these workflows are going to improve the state of the framework in that regard. In the context of CliWrap, many of the issues mentioned (deadlocks, cancellation, kill on parent death, etc) have already been solved either by leveraging the existing Process APIs or by native system calls. However, there are some things that CliWrap could benefit from, namely:
Unfortunately, it would still be troublesome to use these new additions in CliWrap because, as a library, it would need to remain compatible with older targets. So realistically, it will probably continue using its own abstractions/workarounds, except for those that can't be backported by reasonable means (which would only be available for users on . NET vNext). As for API naming/design suggestions, I'm afraid that my suggestions would just devolve toward mimicking CliWrap, as I think that's the best design for spawning processes there is 😄 Let me know if I fully answered your questions |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
Hi @Tyrrrz,
My name is Adam, I'm working on a set of new APIs for .NET to address many of the pain points with System.Diagnostics.Process. Since your project makes heavy use of process execution, I'd greatly appreciate your feedback on the design.
Overview
I'm proposing a layered API approach:
Key benefits:
The full proposal and working implementation is available at: https://github.com/adamsitnik/ProcessPlayground
Request for Feedback
I'd love to hear your thoughts on:
Thanks for considering this, and I look forward to your feedback!
Best regards,
Adam
Beta Was this translation helpful? Give feedback.
All reactions