Skip to content

Commit

Permalink
Fix node-gyp building on MacOS
Browse files Browse the repository at this point in the history
  • Loading branch information
rubensworks committed Aug 29, 2024
1 parent fcb86bc commit 565ccde
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 3 deletions.
12 changes: 10 additions & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,16 @@ jobs:
test:
runs-on: ${{ matrix.os }}
strategy:
fail-fast: false
matrix:
os: [ubuntu-latest, macos-latest, windows-latest]
os:
- ubuntu-20.04
- ubuntu-22.04
- ubuntu-latest
- macos-12
- macos-13
- macos-14
- macos-latest
node-version:
- 18.x
- 20.x
Expand All @@ -34,7 +42,7 @@ jobs:
**/node_modules
.rdf-test-suite-cache
.rdf-test-suite-ldf-cache
key: ${{ runner.os }}-test-modules-${{ hashFiles('**/package-lock.json') }}
key: ${{ runner.os }}-${{ runner.node-version }}-test-modules-${{ hashFiles('**/package-lock.json') }}
- name: Install dependencies
run: npm install
- name: Run tests
Expand Down
3 changes: 2 additions & 1 deletion binding.gyp
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,7 @@
],
"defines": [
"HAVE_CDS",
"_LIBCPP_ENABLE_CXX17_REMOVED_UNARY_BINARY_FUNCTION",
],
"cflags!": [ "-fno-rtti", "-fno-exceptions" ],
"cflags_cc!": [ "-fno-rtti", "-fno-exceptions" ],
Expand All @@ -46,7 +47,7 @@
"GCC_ENABLE_CPP_EXCEPTIONS": "YES",
"OTHER_CFLAGS": [
"-stdlib=libc++",
"-Wno-register"
"-Wno-register",
],
},
},
Expand Down

0 comments on commit 565ccde

Please sign in to comment.