Skip to content

Commit

Permalink
use explicit path for update wasm task
Browse files Browse the repository at this point in the history
  • Loading branch information
vlopes11 committed Nov 15, 2023
1 parent 95304ed commit 2c683f5
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 5 deletions.
4 changes: 1 addition & 3 deletions .github/workflows/build-lint-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -86,14 +86,12 @@ jobs:
PACKAGE_URL="https://github.com/WebAssembly/binaryen/releases/download/version_116/binaryen-version_116-x86_64-linux.tar.gz"
INSTALL_PATH="/usr/local/bin"
curl -L $PACKAGE_URL | tar xz -C $INSTALL_PATH
echo "export PATH=$INSTALL_PATH/binaryen-version_116/bin:\$PATH" >> $GITHUB_ENV
shell: bash
- name: Install WABT
run: |
PACKAGE_URL="https://github.com/WebAssembly/wabt/releases/download/1.0.34/wabt-1.0.34-ubuntu.tar.gz"
INSTALL_PATH="/usr/local/bin"
curl -L $PACKAGE_URL | tar xz -C $INSTALL_PATH
echo "export PATH=$INSTALL_PATH/wabt-1.0.34/bin:\$PATH" >> $GITHUB_ENV
shell: bash
- name: Setup Rust toolchain
run: rustup target add wasm32-wasi
Expand All @@ -106,7 +104,7 @@ jobs:
- name: Install Google Chrome
run: yarn install-chrome
- name: Update WASM
run: yarn update-wasm
run: PATH="/usr/local/bin/binaryen-version_116/bin:/usr/local/bin/wabt-1.0.34/bin:\$PATH" yarn update-wasm
- name: Run e2e tests
run: yarn workspace sov-snap run test
- name: Require clean working directory
Expand Down
4 changes: 2 additions & 2 deletions external/sov-wasm/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -27,8 +27,8 @@ wasm: ## Build the WASM files
$(WAT) \
$(PACKAGE_RESTORED) \
$(PACKAGE_RESTORED_OPTIMIZED)
@mv $(PACKAGE_THIN) $(PACKAGE)
@cp $(PACKAGE) $(PACKAGE_SHORT)
@mv $(PACKAGE_THIN) $(PACKAGE_SHORT)
@rm $(PACKAGE)
@echo "Package created: $(PACKAGE)"

help: ## Display this help screen
Expand Down
Binary file removed external/sov-wasm/assets/sov-wasm-0.1.0.wasm
Binary file not shown.

0 comments on commit 2c683f5

Please sign in to comment.