Wasmtime GitHub notifications bot (Jul 24 2023 at 23:49): Wasmtime GitHub notifications bot (Jul 24 2023 at 23:49):elliottt opened PR #6764 from bytecodealliance:trevor/stdio-stream-testing to bytecodealliance:main:
wip
<!--
Please make sure you include the following information:
If this work has been discussed elsewhere, please include a link to that
conversation. If it was discussed in an issue, just mention "issue #...".Explain why this change is needed. If the details are in an issue already,
this can be brief.Our development process is documented in the Wasmtime book:
https://docs.wasmtime.dev/contributing-development-process.htmlPlease ensure all communication follows the code of conduct:
https://github.com/bytecodealliance/wasmtime/blob/main/CODE_OF_CONDUCT.md
-->
Wasmtime GitHub notifications bot (Jul 24 2023 at 23:51):pchickey updated PR #6764.
Wasmtime GitHub notifications bot (Jul 24 2023 at 23:52):elliottt updated PR #6764.
Wasmtime GitHub notifications bot (Jul 24 2023 at 23:53):elliottt edited PR #6764:
Add a helper for writing tests of wasi-streams that rely on piped stdin, and re-implement the worker thread stdin implementation for windows.
<!--
Please make sure you include the following information:
If this work has been discussed elsewhere, please include a link to that
conversation. If it was discussed in an issue, just mention "issue #...".Explain why this change is needed. If the details are in an issue already,
this can be brief.Our development process is documented in the Wasmtime book:
https://docs.wasmtime.dev/contributing-development-process.htmlPlease ensure all communication follows the code of conduct:
https://github.com/bytecodealliance/wasmtime/blob/main/CODE_OF_CONDUCT.md
-->
Wasmtime GitHub notifications bot (Jul 25 2023 at 00:35):pchickey updated PR #6764.
Wasmtime GitHub notifications bot (Jul 25 2023 at 00:36):elliottt updated PR #6764.
Wasmtime GitHub notifications bot (Jul 25 2023 at 00:37):pchickey updated PR #6764.
Wasmtime GitHub notifications bot (Jul 25 2023 at 17:38):pchickey updated PR #6764.
Wasmtime GitHub notifications bot (Jul 26 2023 at 00:00):pchickey updated PR #6764.
Wasmtime GitHub notifications bot (Jul 26 2023 at 00:02):pchickey updated PR #6764.
Wasmtime GitHub notifications bot (Jul 26 2023 at 18:34):elliottt updated PR #6764.
Wasmtime GitHub notifications bot (Jul 26 2023 at 18:34):elliottt updated PR #6764.
Wasmtime GitHub notifications bot (Jul 27 2023 at 22:27):elliottt updated PR #6764.
Wasmtime GitHub notifications bot (Jul 27 2023 at 22:27):elliottt updated PR #6764.
Wasmtime GitHub notifications bot (Jul 27 2023 at 22:42):elliottt edited PR #6764:
Test the stdio implementations in the wasi crate, which yielded a few needed changes:
AsyncReadStreamandAsyncWriteStreamboth now terminate the associated tokio::task when they are dropped.- The
stdinimplementation on unix and windows now properly handles spanning tokio runtime restarts, and process termination.After implementing these changes, the wasi implementation of
stdinworks on windows now and is better tested on unix.<!--
Please make sure you include the following information:
If this work has been discussed elsewhere, please include a link to that
conversation. If it was discussed in an issue, just mention "issue #...".Explain why this change is needed. If the details are in an issue already,
this can be brief.Our development process is documented in the Wasmtime book:
https://docs.wasmtime.dev/contributing-development-process.htmlPlease ensure all communication follows the code of conduct:
https://github.com/bytecodealliance/wasmtime/blob/main/CODE_OF_CONDUCT.md
-->
Wasmtime GitHub notifications bot (Jul 27 2023 at 22:43):elliottt edited PR #6764:
Test the stdio implementations in the wasi crate, by forking a child process and controlling its stdin directly. This testing exposed a few needed changes:
AsyncReadStreamandAsyncWriteStreamboth now terminate the associated tokio::task when they are dropped.- The
stdinimplementation on unix and windows now properly handles spanning tokio runtime restarts, and process termination.After implementing these changes, the wasi implementation of
stdinworks on windows now and is better tested on unix.<!--
Please make sure you include the following information:
If this work has been discussed elsewhere, please include a link to that
conversation. If it was discussed in an issue, just mention "issue #...".Explain why this change is needed. If the details are in an issue already,
this can be brief.Our development process is documented in the Wasmtime book:
https://docs.wasmtime.dev/contributing-development-process.htmlPlease ensure all communication follows the code of conduct:
https://github.com/bytecodealliance/wasmtime/blob/main/CODE_OF_CONDUCT.md
-->
Wasmtime GitHub notifications bot (Jul 27 2023 at 22:43):elliottt has marked PR #6764 as ready for review.
Wasmtime GitHub notifications bot (Jul 27 2023 at 22:43):elliottt requested fitzgen for a review on PR #6764.
Wasmtime GitHub notifications bot (Jul 27 2023 at 22:43):elliottt requested wasmtime-core-reviewers for a review on PR #6764.
Wasmtime GitHub notifications bot (Jul 27 2023 at 22:44):elliottt updated PR #6764.
Wasmtime GitHub notifications bot (Jul 27 2023 at 22:45):elliottt requested pchickey for a review on PR #6764.
Wasmtime GitHub notifications bot (Jul 27 2023 at 22:45):pchickey submitted PR review.
Wasmtime GitHub notifications bot (Jul 27 2023 at 22:45):pchickey has enabled auto merge for PR #6764.
Wasmtime GitHub notifications bot (Jul 28 2023 at 17:39):pchickey updated PR #6764.
Wasmtime GitHub notifications bot (Jul 31 2023 at 21:50):elliottt updated PR #6764.
Wasmtime GitHub notifications bot (Jul 31 2023 at 23:12):pchickey merged PR #6764.
Last updated: Nov 22 2024 at 16:03 UTC