Skip to content

Commit

Permalink
Update emscripten requirements and WASM CI job
Browse files Browse the repository at this point in the history
  • Loading branch information
iamnotacake committed Nov 10, 2023
1 parent 05cac26 commit 362e31c
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 4 deletions.
8 changes: 5 additions & 3 deletions .github/workflows/test-wasm.yaml
Expand Up @@ -79,9 +79,11 @@ jobs:
matrix:
node-version:
- 10.x # legacy
- 12.x # old LTS
- 14.x # current LTS
- 16.x # current stable
- 12.x # legacy
- 14.x # legacy
- 16.x # legacy
- 18.x # current LTS
- 20.x # current active
fail-fast: false
steps:
- name: Install Node.js ${{ matrix.node-version }}
Expand Down
2 changes: 1 addition & 1 deletion src/wrappers/themis/wasm/emscripten/VERSION
@@ -1 +1 @@
3.0.0
3.1.47
1 change: 1 addition & 0 deletions src/wrappers/themis/wasm/wasmthemis.mk
Expand Up @@ -28,6 +28,7 @@ $(BIN_PATH)/libthemis.js: LDFLAGS += -s EXPORTED_RUNTIME_METHODS=@$(WASM_RUNTIME
$(BIN_PATH)/libthemis.js: LDFLAGS += -s ALLOW_TABLE_GROWTH
$(BIN_PATH)/libthemis.js: LDFLAGS += -s MODULARIZE=1
$(BIN_PATH)/libthemis.js: LDFLAGS += -s ALLOW_MEMORY_GROWTH=1
$(BIN_PATH)/libthemis.js: LDFLAGS += -s EXPORTED_FUNCTIONS='["_malloc", "_free"]'
# FIXME(ilammy, 2020-11-29): rely in EMSCRIPTEN_KEEPALIVE instead of LINKABLE
# For some reason existing EMSCRIPTEN_KEEPALIVE macros do not work and without
# LINKABLE flag wasm-ld ends up stripping *all* Themis functions from "*.wasm"
Expand Down

0 comments on commit 362e31c

Please sign in to comment.