Wasmtime GitHub notifications bot (May 20 2021 at 23:12): Wasmtime GitHub notifications bot (May 20 2021 at 23:12):pchickey edited PR #2917 from pch/wasmtime_wasi_reexport_mess to main.
Wasmtime GitHub notifications bot (May 20 2021 at 23:13):pchickey edited PR #2917 from pch/wasmtime_wasi_reexport_mess to main:
wasi-common's WasiCtxBuilder was really only useful wasi_cap_std_sync and
wasi_tokio to implement their own Builder on top of. But even so, wasi_common::WasiCtxBuilder was barely better than just manipulating a WasiCtx directly. So now the sync and tokio builders do so directly.wasmtime-wasi was re-exporting wasi-common::WasiCtxBuilder is 1. not useful and 2. shadow's the
re-export of the right one in sync::*.<!--
Please ensure that the following steps are all taken care of before submitting
the PR.
[ ] This has been discussed in issue #..., or if not, please tell us why
here.[ ] A short description of what this does, why it is needed; if the
description becomes long, the matter should probably be discussed in an issue
first.[ ] This PR contains test cases, if meaningful.
- [ ] A reviewer from the core maintainer team has been assigned for this PR.
If you don't know who could review this, please indicate so. The list of
suggested reviewers on the right can help you.Please ensure all communication adheres to the code of conduct.
-->
Wasmtime GitHub notifications bot (May 20 2021 at 23:13):pchickey edited PR #2917 from pch/wasmtime_wasi_reexport_mess to main:
wasi-common's WasiCtxBuilder was really only useful wasi_cap_std_sync and wasi_tokio to implement their own Builder on top of. But even so, wasi_common::WasiCtxBuilder was barely better than just manipulating a WasiCtx directly. So now the sync and tokio builders do so directly.
wasmtime-wasi was re-exporting wasi-common::WasiCtxBuilder is 1. not useful and 2. shadow's the
re-export of the right one in sync::*.<!--
Please ensure that the following steps are all taken care of before submitting
the PR.
[ ] This has been discussed in issue #..., or if not, please tell us why
here.[ ] A short description of what this does, why it is needed; if the
description becomes long, the matter should probably be discussed in an issue
first.[ ] This PR contains test cases, if meaningful.
- [ ] A reviewer from the core maintainer team has been assigned for this PR.
If you don't know who could review this, please indicate so. The list of
suggested reviewers on the right can help you.Please ensure all communication adheres to the code of conduct.
-->
Wasmtime GitHub notifications bot (May 20 2021 at 23:13):pchickey requested alexcrichton for a review on PR #2917.
Last updated: Nov 22 2024 at 16:03 UTC