-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
"pub run" should spawn an isolate rather than starting a subprocess #21821
Comments
Added Pub-Run label. |
This is blocking dart-lang/test#19. |
This comment was originally written by @seaneagan In the meantime could This is also blocking https://github.com/google/stagehand/issues/147 |
This comment was originally written by @seaneagan This would fix |
Platform.script is already not what people expect under "pub run". Ideally there would be a way to spawn an isolate with a custom value for Platform.script, but that feature request is out-of-scope for this issue. |
This comment was originally written by @seaneagan I think Platform.script currently references the pub cache location of the script, no? I only need (BTW my use case is for unscripted to determine the command name for help text output.) |
This comment was originally written by @seaneagan See also issue #22098, which would be an even better solution for my use case. |
This is also blocking dart-lang/test#86. |
This comment was originally written by emil.n.pe...@gmail.com Case where this is a real problem: |
This issue has been moved to dart-lang/pub#1204. |
Currently "pub run" starts a subprocess to run its code. It should instead spawn an isolate to avoid VM startup time and provide accurate information for the stdioType getter.
The text was updated successfully, but these errors were encountered: