Skip to content

Commit

Permalink
also build on windows
Browse files Browse the repository at this point in the history
  • Loading branch information
jraymakers committed Jun 30, 2024
1 parent e2190dc commit 6db6d57
Show file tree
Hide file tree
Showing 2 changed files with 34 additions and 6 deletions.
25 changes: 20 additions & 5 deletions .github/workflows/NodeJS.yml
Original file line number Diff line number Diff line change
Expand Up @@ -198,9 +198,9 @@ jobs:
# shell: bash
# run: ./scripts/node_build.sh ${{ matrix.node }}

# win-nodejs:
# name: node.js Windows
# runs-on: windows-latest
win-nodejs:
name: node.js Windows
runs-on: windows-latest
# needs: linux-nodejs
# continue-on-error: ${{ matrix.node != '18' && matrix.node != '20' && matrix.node != '21' }}
# env:
Expand All @@ -227,12 +227,27 @@ jobs:
# - isRelease: false
# node: 20

# steps:
steps:
# - uses: actions/setup-python@v4
# with:
# python-version: '3.8'

# - uses: actions/checkout@v3
- uses: actions/checkout@v4
with:
sparse-checkout: alt

- name: Bindings (Install, Configure, and Build)
working-directory: alt/bindings
run: npm i

- name: Test (Install)
working-directory: alt/test
run: npm i

- name: Test (Run)
working-directory: alt/test
run: npm run test

# with:
# fetch-depth: 0

Expand Down
15 changes: 14 additions & 1 deletion alt/bindings/binding.gyp
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,20 @@
'destination': 'package/lib',
},
],
}]
}],
['OS=="win"', {
'link_settings': {
'libraries': [
'<(module_root_dir)/libduckdb/libduckdb.lib',
],
},
'copies': [
{
'files': ['<(module_root_dir)/libduckdb/libduckdb.dll'],
'destination': 'package/lib',
},
],
}],
]
},
{
Expand Down

0 comments on commit 6db6d57

Please sign in to comment.