Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add subprocess support #9

Closed
agronholm opened this issue Oct 12, 2018 · 1 comment · Fixed by #134
Closed

Add subprocess support #9

agronholm opened this issue Oct 12, 2018 · 1 comment · Fixed by #134
Labels
enhancement New feature or request
Milestone

Comments

@agronholm
Copy link
Owner

agronholm commented Oct 12, 2018

There should be some support for subprocesses, at least on the level that all the backends provide.

Two kinds of use cases can be seen for subprocesses:

  1. Running external commands
  2. Running CPU-bound functions

The first case would probably be easier to implement, but eventually we need both.

@agronholm agronholm added the enhancement New feature or request label Oct 12, 2018
@agronholm agronholm added this to the 1.1.0 milestone Nov 14, 2018
@agronholm agronholm removed this from the 1.1.0 milestone Aug 7, 2019
@agronholm agronholm added this to the 1.2.0 milestone Sep 9, 2019
@agronholm
Copy link
Owner Author

This one is going to be a problem because of Windows. The initial implementation works fine on Linux and macOS but on Windows there are a couple of big problems:

  • Curio tries to call a nonexistent (on Windows) os.set_blocking() function
  • Asyncio needs the IOCP based proactor event loop (which makes all anyio network stuff fail)

To make this works properly on asyncio, we would need to use its native Protocol based I/O instead of add_reader() and remove_reader().

@agronholm agronholm modified the milestones: 1.2.0, 2.0.0 Oct 3, 2019
agronholm added a commit that referenced this issue Dec 30, 2019
agronholm added a commit that referenced this issue Jul 30, 2020
agronholm added a commit that referenced this issue Jul 30, 2020
agronholm added a commit that referenced this issue Jul 31, 2020
agronholm added a commit that referenced this issue Aug 1, 2020
agronholm added a commit that referenced this issue Aug 2, 2020
mjwestcott pushed a commit to mjwestcott/anyio that referenced this issue Aug 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant