Skip to content

Commit 2627620

Browse files
committed
fix: update WASM build instructions and exclude wasm.c from coverage
1 parent d042fe2 commit 2627620

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

.github/workflows/main.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ permissions:
1111
jobs:
1212
build:
1313
runs-on: ${{ matrix.os }}
14-
name: ${{ matrix.name }}${{ matrix.arch && format('-{0}', matrix.arch) || '' }} build${{ matrix.arch != 'arm64-v8a' && matrix.name != 'isim' && matrix.name != 'ios' && ' + test' || ''}}
14+
name: ${{ matrix.name }}${{ matrix.arch && format('-{0}', matrix.arch) || '' }} build${{ matrix.arch != 'arm64-v8a' && matrix.name != 'isim' && matrix.name != 'ios' && matrix.name != 'wasm' && ' + test' || ''}}
1515
timeout-minutes: 20
1616
strategy:
1717
fail-fast: false
@@ -74,7 +74,7 @@ jobs:
7474

7575
- name: wasm install wabt
7676
if: matrix.name == 'wasm'
77-
run: apt install wabt
77+
run: sudo apt install wabt
7878

7979
- name: build sqlite-sync
8080
run: make extension ${{ matrix.make && matrix.make || ''}}

Makefile

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ TEST_SRC = $(wildcard $(TEST_DIR)/*.c)
5757
TEST_FILES = $(SRC_FILES) $(TEST_SRC) $(wildcard $(SQLITE_DIR)/*.c)
5858
RELEASE_OBJ = $(patsubst %.c, $(BUILD_RELEASE)/%.o, $(notdir $(SRC_FILES)))
5959
TEST_OBJ = $(patsubst %.c, $(BUILD_TEST)/%.o, $(notdir $(TEST_FILES)))
60-
COV_FILES = $(filter-out $(SRC_DIR)/lz4.c $(SRC_DIR)/network.c, $(SRC_FILES))
60+
COV_FILES = $(filter-out $(SRC_DIR)/lz4.c $(SRC_DIR)/network.c $(SRC_DIR)/wasm.c, $(SRC_FILES))
6161
CURL_LIB = $(CURL_DIR)/$(PLATFORM)/libcurl.a
6262
TEST_TARGET = $(patsubst %.c,$(DIST_DIR)/%$(EXE), $(notdir $(TEST_SRC)))
6363

@@ -306,7 +306,7 @@ help:
306306
@echo " android (needs ARCH to be set to x86_64 or arm64-v8a and ANDROID_NDK to be set)"
307307
@echo " ios (only on macOS)"
308308
@echo " isim (only on macOS)"
309-
@echo " wasm (needs wabt[brew install wabt/apt install wabt])"
309+
@echo " wasm (needs wabt[brew install wabt/sudo apt install wabt])"
310310
@echo ""
311311
@echo "Targets:"
312312
@echo " all - Build the extension (default)"

0 commit comments

Comments
 (0)