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

Fix installing wasmer via cargo-binstall #3369

Closed
wants to merge 5 commits into from

Conversation

NobodyXu
Copy link
Contributor

@NobodyXu NobodyXu commented Nov 24, 2022

Signed-off-by: Jiahao XU [email protected]

Description

With wasmer v3.0.1, the link to release artifacts changed a little bit which breaks installing wasmer via cargo-binstall.
Also the instruction for installing wasmer via cargo-binstall in README.md is wrong.

Review

  • Add a short description of the change to the CHANGELOG.md file

@fschutt fschutt mentioned this pull request Nov 24, 2022
@fschutt
Copy link
Contributor

fschutt commented Nov 24, 2022

There were some network outages on wapm.dev which is why this PR didn't pass immediately. I reopened + followed up on this in #3371 (it's easier to merge that way). Thank you for your contribution!

@fschutt fschutt closed this Nov 24, 2022
bors bot added a commit that referenced this pull request Nov 24, 2022
3371: Fix cargo binstall r=fschutt a=fschutt

This PR is a continuation of #3369 because for some unexpected reason wasmer login failed, so it's easier for me to debug the CI failure.

Co-authored-by: Jiahao XU <[email protected]>
Co-authored-by: Felix Schütt <[email protected]>
Co-authored-by: Felix Schütt <[email protected]>
@NobodyXu NobodyXu deleted the fix/cargo-binstall branch November 24, 2022 23:53
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.

2 participants