Skip to content

Commit adbbf2c

Browse files
seclerpabrown
andauthored
Fix installation and usage instructions in README.md (#488)
* Add missing architecture URL part to the installation script * Remove unnecessary `export`s from installation instructions * Fix missing $ sign in export * Add `WASI_OS` as new parameter to use & install scripts in README --------- Co-authored-by: Andrew Brown <[email protected]>
1 parent 754aec3 commit adbbf2c

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

README.md

+7-5
Original file line numberDiff line numberDiff line change
@@ -132,18 +132,20 @@ see [RELEASING.md](RELEASING.md).
132132
A typical installation from the release binaries might look like the following:
133133

134134
```shell script
135-
export WASI_VERSION=20
136-
export WASI_VERSION_FULL=${WASI_VERSION}.0
137-
wget https://github.com/WebAssembly/wasi-sdk/releases/download/wasi-sdk-${WASI_VERSION}/wasi-sdk-${WASI_VERSION_FULL}-linux.tar.gz
138-
tar xvf wasi-sdk-${WASI_VERSION_FULL}-linux.tar.gz
135+
WASI_OS=linux
136+
WASI_ARCH=x86_64
137+
WASI_VERSION=24
138+
WASI_VERSION_FULL=${WASI_VERSION}.0
139+
wget https://github.com/WebAssembly/wasi-sdk/releases/download/wasi-sdk-${WASI_VERSION}/wasi-sdk-${WASI_VERSION_FULL}-${WASI_ARCH}-${WASI_OS}.tar.gz
140+
tar xvf wasi-sdk-${WASI_VERSION_FULL}-${WASI_ARCH}-${WASI_OS}.tar.gz
139141
```
140142

141143
## Use
142144

143145
Use the clang installed in the `wasi-sdk` directory:
144146

145147
```shell script
146-
export WASI_SDK_PATH=`pwd`/wasi-sdk-${WASI_VERSION_FULL}
148+
WASI_SDK_PATH=`pwd`/wasi-sdk-${WASI_VERSION_FULL}-${WASI_ARCH}-${WASI_OS}
147149
CC="${WASI_SDK_PATH}/bin/clang --sysroot=${WASI_SDK_PATH}/share/wasi-sysroot"
148150
$CC foo.c -o foo.wasm
149151
```

0 commit comments

Comments
 (0)