Skip to content

Conversation

@scriptjs
Copy link

These updates should help others get started.

@xtuc
Copy link
Member

xtuc commented Oct 15, 2018

r? @sendilkumarn @ashleygwilliams


### `wasm-bindgen-cli`

`wasm-bindgen` is a build requirement for `wasm-pack` and a dependency of `wasm-bindgen-cli`. To verify installation, run:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

But wont wasm-pack installs this automatically from v0.5


We expect `wasm-pack` to be in your `$PATH`. See [installation here](https://github.com/rustwasm/wasm-pack/blob/master/docs/src/setup.md#installing-wasm-pack).

### `wasm-bindgen-cli`
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@fitzgen mentioned that this isn't needed. Could you please take a look @fitzgen

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Using the >= 0.5.0 versions of wasm-pack will automatically install the wasm-bindgen CLI for you (and the right version!) if necessary, so it shouldn't be a prerequisite to already have it installed anymore.

Just sent #36 too since we generally don't want to encourage -m no-install.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure what you mean, I ran into issue #30 with 0.5.1 of wasm-pack.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That was because no-install mode was used, which forces wasm-pack into not downloading the w32-u-u target

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There have been commits since then that removed the use of no-install, btw.

@cktang88
Copy link

Can we get this merged?

@fitzgen
Copy link
Contributor

fitzgen commented Feb 25, 2019

Can we get this merged?

The wasm-bindgen-cli section should be removed. Then it should be good to go!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants