brson / wasm-opt-rs

Rust bindings for Binaryen's wasm-opt
Apache License 2.0
61 stars 10 forks source link

fix: allow compiling to wasm32-wasi #165

Closed fibonacci1729 closed 5 months ago

fibonacci1729 commented 5 months ago

This fixes a type mismatch when compiling wasm-opt targeting wasm32-wasi.

error[E0308]: mismatched types
   --> wasm-opt-0.116.0/src/base.rs:397:8
    |
397 |     Ok(path)
    |     -- ^^^^ expected `&[u8]`, found `&Path`
    |     |
    |     arguments to this enum variant are incorrect
    |
    = note: expected reference `&[u8]`
               found reference `&Path`

cc @alexcrichton

alexcrichton commented 5 months ago

👋 @brson long time no see!

brson commented 5 months ago

Thanks @fibonacci1729 @alexcrichton I'll aim to get this into a point release this weekend.

brson commented 5 months ago

Looks like the CI failure is unrelated.

brson commented 5 months ago

This is published as 0.116.1