Skip to content

Add comment for ARM64 package installation to README.md#515

Merged
alexcrichton merged 2 commits intoWebAssembly:mainfrom
yomaytk:fix-readme
Feb 17, 2025
Merged

Add comment for ARM64 package installation to README.md#515
alexcrichton merged 2 commits intoWebAssembly:mainfrom
yomaytk:fix-readme

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