Skip to content

Commit

Permalink
fix emscripten.yml workflow
Browse files Browse the repository at this point in the history
fix formatting in some commands. Add missing -fsanitize=address to the ASAN test
  • Loading branch information
MoustaphaSaad authored and botovq committed Jun 19, 2024
1 parent d661f77 commit 6f37c93
Showing 1 changed file with 7 additions and 4 deletions.
11 changes: 7 additions & 4 deletions .github/workflows/emscripten.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,10 +25,10 @@ jobs:
- name: "Setup emsdk"
uses: mymindstorm/setup-emsdk@v14
with:
version: 3.1.60
version: "3.1.60"

- name: "Prepare repository"
run: "./autogen.sh"
run: ./autogen.sh

- name: "Configure CMake"
run: emcmake cmake -Bbuild
Expand All @@ -53,13 +53,16 @@ jobs:
- name: "Setup emsdk"
uses: mymindstorm/setup-emsdk@v14
with:
version: 3.1.60
version: "3.1.60"

- name: "Prepare repository"
run: "./autogen.sh"
run: ./autogen.sh

- name: "Configure CMake"
run: emcmake cmake -Bbuild
env:
CFLAGS: "-ggdb -fsanitize=address"
LDFLAGS: "-fsanitize=address"

- name: "Build"
run: cmake --build build --config Release
Expand Down

0 comments on commit 6f37c93

Please sign in to comment.