Skip to content

Conversation

yomaytk
Copy link
Contributor

@yomaytk yomaytk commented Feb 15, 2025

Hi! I added WASI_ARCH=arm64 command for the arm64 package version to the installation command’s comment. I thought this would make the command more clear.

@yomaytk yomaytk changed the title Add comment for installation command of README.md Add comment for ARM64 package installation to README.md Feb 15, 2025
README.md Outdated
```shell script
WASI_OS=linux
WASI_ARCH=x86_64
WASI_ARCH=x86_64 // (`WASI_ARCH=arm64` for ARM64 package)
Copy link
Member

Choose a reason for hiding this comment

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

How about # or 'arm64' if running on arm64 host

(# is the command char in the scripts)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@sbc100
Thank you for the review. I fixed a comment.

@alexcrichton alexcrichton enabled auto-merge (squash) February 17, 2025 12:12
@alexcrichton alexcrichton merged commit 5a3567a into WebAssembly:main Feb 17, 2025
8 checks passed
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.

3 participants