New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
wasm-bindgen-cli: init at 0.2.51 #67722
Conversation
f71d2b2
to
63fa6e0
Compare
@mmahut fixed :) |
63fa6e0
to
54413f7
Compare
@GrahamcOfBorg build wasm-bindgen-cli |
0dc0aec
to
afe6a95
Compare
Disabled darwin build for now as I have no time and interest in maintaining a stable darwin build for this package as well. In case somebody is interested in darwin suport, feel free to file a patch :) |
That's fine 👍 As the package maintainer it's best to support what you actually can support. |
...pment/tools/wasm-bindgen-cli/0001-Add-cargo.lock-for-rustPlatform.buildRustPackage-in-.patch
Show resolved
Hide resolved
I made a patch to support macOS builds, if you want to merge it: Ma27/nixpkgs@package-wasm-bindgen-cli...B4dM4n:wasm-bindgen-cli-darwin Also note, when building the package I got a |
I dug a bit deeper and the issue probably arises from the fact that So whenever the inputs of |
Yeah, this is one of the downsides of this approach with fixed-output derivations. I'm currently investigating a different issue with wasm-bindgen-cli 0.2.50 which is why this is a bit delayed now... Will report back as soon as I have something usable :) |
ebe4fbb
to
8dc6919
Compare
Hmm, it actually seems as the cargo hash isn't deterministic (I actually rechecked it locally). Let's see what the x86_64-linux and darwin build does. Will take a look into this. |
Aah.... it seems as some cargo-vendor changes were made, this branch is rather old. After rebasing onto latest master I get the same hash now. Currently rebuilding, will push a fix then :) |
CLI for `wasm-bindgen`, a tool that helps to interact between WASM modules written in Rust and JS-Code. This contains only the CLI[1], the libraries are usually included via `Cargo.toml` in a project. Co-authored-by: Fabian Möller <fabianm88@gmail.com> [1] https://crates.io/crates/wasm-bindgen-cli
8dc6919
to
9bffa5a
Compare
@ofborg build wasm-bindgen-cli |
Motivation for this change
CLI for
wasm-bindgen
, a tool that helps to interact between WASMmodules written in Rust and JS-Code. This contains only the CLI[1], the
libraries are usually included via
Cargo.toml
in a project.[1] https://crates.io/crates/wasm-bindgen-cli
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @