Skip to content
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

Content bug: Rust_to_wasm fails to mention Perl and Make #7831

Closed
Dionysusnu opened this issue Aug 11, 2021 · 3 comments
Closed

Content bug: Rust_to_wasm fails to mention Perl and Make #7831

Dionysusnu opened this issue Aug 11, 2021 · 3 comments
Labels
Content:wasm WebAssembly docs

Comments

@Dionysusnu
Copy link

Dionysusnu commented Aug 11, 2021

What page(s) did you find the problem on?

Compiling from Rust to WebAssembly
https://developer.mozilla.org/en-US/docs/WebAssembly/Rust_to_wasm

Specific page section or heading?

https://developer.mozilla.org/en-US/docs/WebAssembly/Rust_to_wasm#wasm-pack

What is the problem?

Installing wasm-pack using cargo requires both perl and make to be installed. These aren't on Windows by default, but nothing is mentioned about it in the guide.

What did you expect to see?

The guide should make mention of this, and recommend how to install both of them. It's a lengthy process though..

Did you test this? If so, how?

I ran into this issue myself, while following the guide.
The error is

error: failed to run custom build command for `openssl-sys v0.9.65`

with some more clutter output lines.
However, no clear problem can be identified from this output. It does mention running a perl script, but it is not at all obvious that it fails because it cannot find perl itself.
This issue for the library shows how unclear the error is: sfackler/rust-openssl#1455

@Dionysusnu Dionysusnu added the needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. label Aug 11, 2021
@Dionysusnu Dionysusnu changed the title Content bug: <TITLE OF PROBLEM> Content bug: Rust_to_wasm fails to mention Perl and Make Aug 11, 2021
@sideshowbarker sideshowbarker added Content:wasm WebAssembly docs and removed needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. labels Aug 13, 2021
himanshugarg pushed a commit to himanshugarg/content that referenced this issue Sep 24, 2021
@himanshugarg
Copy link
Contributor

Closed the PR (#9254) as Rust setup docs are more appropriate for documenting this.

@wbamberg
Copy link
Collaborator

Yes, closing this, because listing requirements for Rust installation are out of scope for this article.

@himanshugarg
Copy link
Contributor

@Dionysusnu thanks for reporting the issue. Please consider posting it on https://github.com/rust-lang/rust-forge or a more appropriate Rust forum for this.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Content:wasm WebAssembly docs
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants