Some custom WASI SDK fixes (#433)

Makefile

@@ -1,9 +1,6 @@
 .PHONY: cli core test fmt clean
 .DEFAULT_GOAL := cli
 
-download-wasi-sdk:
-	./install-wasi-sdk.sh
-
 install:
 	cargo install --path crates/cli
 

crates/quickjs-wasm-rs/Cargo.toml

@@ -11,7 +11,7 @@ categories = ["api-bindings"]
 
 [dependencies]
 anyhow = { workspace = true }
-quickjs-wasm-sys = { version = "1.0.0", path = "../quickjs-wasm-sys" }
+quickjs-wasm-sys = { version = "1.1.0-alpha.1", path = "../quickjs-wasm-sys" }
 serde = { version = "1.0", features = ["derive"] }
 once_cell = "1.16"
 

crates/quickjs-wasm-sys/CHANGELOG.md

@@ -7,6 +7,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## Unreleased
 
+- Added: `QUICKJS_WASM_SYS_WASI_SDK_MAJOR_VERSION` and `QUICKJS_WASM_SYS_WASI_SDK_MINOR_VERSION` build-time environment variables to control which version of the WASI SDK to use.
+- Fixed: Changing the `QUICKJS_WASM_SYS_WASI_SDK_PATH` build time environment variable will trigger a rebuild of the crate.
+
 ## 1.0.0 - 2023-05-16
 
 No changes from 0.1.2. Just updating version to show we're confident in the existing bindings.

crates/quickjs-wasm-sys/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "quickjs-wasm-sys"
-version = "1.0.0"
+version = "1.1.0-alpha.1"
 authors.workspace = true
 edition.workspace = true 
 license.workspace = true

crates/quickjs-wasm-sys/README.md

@@ -8,4 +8,4 @@ To publish this crate to crates.io, run `./publish.sh`.
 
 ## Using a custom WASI SDK
 
-This crate can be compiled using a custom [WASI SDK](https://github.com/WebAssembly/wasi-sdk). When building this crate, set the `QUICKJS_WASM_SYS_WASI_SDK_PATH` environment variable to the absolute path where you installed the SDK.
+This crate can be compiled using a custom [WASI SDK](https://github.com/WebAssembly/wasi-sdk). When building this crate, set the `QUICKJS_WASM_SYS_WASI_SDK_PATH` environment variable to the absolute path where you installed the SDK. You can also use a particular version of the WASI SDK by setting the `QUICKJS_WASM_SYS_WASI_SDK_MAJOR_VERSION` and `QUICKJS_WASM_SYS_WASI_SDK_MINOR_VERSION` environment variables to the appropriate versions.

crates/quickjs-wasm-sys/build.rs

@@ -70,8 +70,17 @@ async fn download_wasi_sdk() -> Result<PathBuf> {
 
     fs::create_dir_all(&wasi_sdk_dir)?;
 
+    const MAJOR_VERSION_ENV_VAR: &str = "QUICKJS_WASM_SYS_WASI_SDK_MAJOR_VERSION";
+    const MINOR_VERSION_ENV_VAR: &str = "QUICKJS_WASM_SYS_WASI_SDK_MINOR_VERSION";
+    println!("cargo:rerun-if-env-changed={MAJOR_VERSION_ENV_VAR}");
+    println!("cargo:rerun-if-env-changed={MINOR_VERSION_ENV_VAR}");
+    let major_version =
+        env::var(MAJOR_VERSION_ENV_VAR).unwrap_or(WASI_SDK_VERSION_MAJOR.to_string());
+    let minor_version =
+        env::var(MINOR_VERSION_ENV_VAR).unwrap_or(WASI_SDK_VERSION_MINOR.to_string());
+
     let mut archive_path = wasi_sdk_dir.clone();
-    archive_path.push("wasi-sdk.tar.gz");
+    archive_path.push(format!("wasi-sdk-{major_version}-{minor_version}.tar.gz"));
 
     // Download archive if necessary
     if !archive_path.try_exists()? {
@@ -83,8 +92,7 @@ async fn download_wasi_sdk() -> Result<PathBuf> {
             other => return Err(anyhow!("Unsupported platform tuple {:?}", other)),
         };
 
-        // FIXME: Make this HTTPS!
-        let uri = format!("https://github.com/WebAssembly/wasi-sdk/releases/download/wasi-sdk-{}/wasi-sdk-{}.{}-{}.tar.gz", WASI_SDK_VERSION_MAJOR, WASI_SDK_VERSION_MAJOR, WASI_SDK_VERSION_MINOR, file_suffix);
+        let uri = format!("https://github.com/WebAssembly/wasi-sdk/releases/download/wasi-sdk-{major_version}/wasi-sdk-{major_version}.{minor_version}-{file_suffix}.tar.gz");
         let mut body = get_uri(uri).await?;
         let mut archive = fs::File::create(&archive_path)?;
         while let Some(frame) = body.frame().await {
@@ -122,7 +130,9 @@ async fn download_wasi_sdk() -> Result<PathBuf> {
 }
 
 async fn get_wasi_sdk_path() -> Result<PathBuf> {
-    if let Ok(path) = env::var("QUICKJS_WASM_SYS_WASI_SDK_PATH") {
+    const WASI_SDK_PATH_ENV_VAR: &str = "QUICKJS_WASM_SYS_WASI_SDK_PATH";
+    println!("cargo:rerun-if-env-changed={WASI_SDK_PATH_ENV_VAR}");
+    if let Ok(path) = env::var(WASI_SDK_PATH_ENV_VAR) {
         return Ok(path.into());
     }
     download_wasi_sdk().await