Remove duplicate add_to_linker calls for preview2 interfaces

bytecodealliance · Sep 27, 2023 · 0ef0823 · 0ef0823
1 parent e9f77e1
commit 0ef0823
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 25 deletions.
diff --git a/crates/test-programs/tests/wasi-http-components-sync.rs b/crates/test-programs/tests/wasi-http-components-sync.rs
@@ -4,7 +4,7 @@ use wasmtime::{
     Config, Engine, Store,
 };
 use wasmtime_wasi::preview2::{
-    command::sync::{add_to_linker, Command},
+    command::sync::Command,
     pipe::MemoryOutputPipe,
     IsATTY, Table, WasiCtx, WasiCtxBuilder, WasiView,
 };
@@ -60,8 +60,7 @@ fn instantiate_component(
     ctx: Ctx,
 ) -> Result<(Store<Ctx>, Command), anyhow::Error> {
     let mut linker = Linker::new(&ENGINE);
-    add_to_linker(&mut linker)?;
-    wasmtime_wasi_http::proxy::add_to_linker(&mut linker)?;
+    wasmtime_wasi_http::proxy::sync::add_to_linker(&mut linker)?;
 
     let mut store = Store::new(&ENGINE, ctx);
 

diff --git a/crates/test-programs/tests/wasi-http-components.rs b/crates/test-programs/tests/wasi-http-components.rs
@@ -4,9 +4,7 @@ use wasmtime::{
     Config, Engine, Store,
 };
 use wasmtime_wasi::preview2::{
-    command::{add_to_linker, Command},
-    pipe::MemoryOutputPipe,
-    IsATTY, Table, WasiCtx, WasiCtxBuilder, WasiView,
+    command::Command, pipe::MemoryOutputPipe, IsATTY, Table, WasiCtx, WasiCtxBuilder, WasiView,
 };
 use wasmtime_wasi_http::{WasiHttpCtx, WasiHttpView};
 
@@ -60,7 +58,6 @@ async fn instantiate_component(
     ctx: Ctx,
 ) -> Result<(Store<Ctx>, Command), anyhow::Error> {
     let mut linker = Linker::new(&ENGINE);
-    add_to_linker(&mut linker)?;
     wasmtime_wasi_http::proxy::add_to_linker(&mut linker)?;
 
     let mut store = Store::new(&ENGINE, ctx);

diff --git a/crates/wasi-http/src/proxy.rs b/crates/wasi-http/src/proxy.rs
@@ -25,15 +25,9 @@ pub fn add_to_linker<T>(l: &mut wasmtime::component::Linker<T>) -> anyhow::Resul
 where
     T: WasiHttpView + preview2::WasiView + bindings::http::types::Host,
 {
-    preview2::bindings::clocks::wall_clock::add_to_linker(l, |t| t)?;
-    preview2::bindings::clocks::monotonic_clock::add_to_linker(l, |t| t)?;
-    preview2::bindings::clocks::timezone::add_to_linker(l, |t| t)?;
-    preview2::bindings::poll::poll::add_to_linker(l, |t| t)?;
-    preview2::bindings::io::streams::add_to_linker(l, |t| t)?;
-    preview2::bindings::random::random::add_to_linker(l, |t| t)?;
-    preview2::bindings::cli::stdout::add_to_linker(l, |t| t)?;
-    preview2::bindings::cli::stderr::add_to_linker(l, |t| t)?;
-    preview2::bindings::cli::stdin::add_to_linker(l, |t| t)?;
+    // TODO: this shouldn't be required, but the adapter unconditionally pulls in all of these
+    // dependencies.
+    preview2::command::add_to_linker(l)?;
 
     bindings::http::outgoing_handler::add_to_linker(l, |t| t)?;
     bindings::http::types::add_to_linker(l, |t| t)?;
@@ -69,15 +63,9 @@ pub mod sync {
     where
         T: WasiHttpView + preview2::WasiView + bindings::http::types::Host,
     {
-        preview2::bindings::clocks::wall_clock::add_to_linker(l, |t| t)?;
-        preview2::bindings::clocks::monotonic_clock::add_to_linker(l, |t| t)?;
-        preview2::bindings::clocks::timezone::add_to_linker(l, |t| t)?;
-        preview2::bindings::io::streams::add_to_linker(l, |t| t)?;
-        preview2::bindings::random::random::add_to_linker(l, |t| t)?;
-        preview2::bindings::random::random::add_to_linker(l, |t| t)?;
-        preview2::bindings::cli::stdout::add_to_linker(l, |t| t)?;
-        preview2::bindings::cli::stderr::add_to_linker(l, |t| t)?;
-        preview2::bindings::cli::stdin::add_to_linker(l, |t| t)?;
+        // TODO: this shouldn't be required, but the adapter unconditionally pulls in all of these
+        // dependencies.
+        preview2::command::sync::add_to_linker(l)?;
 
         bindings::http::outgoing_handler::add_to_linker(l, |t| t)?;
         bindings::http::types::add_to_linker(l, |t| t)?;