Skip to content

fix(NODE-6590): add build for x64 macos #45

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 9 commits into from
Dec 4, 2024
Merged

fix(NODE-6590): add build for x64 macos #45

Show file tree
Hide file tree
Changes from 5 commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
edaaf5e
build universal binary
baileympearson Dec 3, 2024
2ba7b1a
asdf
baileympearson Dec 3, 2024
f31913d
fix lint
baileympearson Dec 3, 2024
375f98f
don't build and test on PRs
baileympearson Dec 3, 2024
7fe1874
errexit
baileympearson Dec 4, 2024
6d9260f
fix builds
baileympearson Dec 4, 2024
b2df3d4
try to fix windows
baileympearson Dec 4, 2024
3ce2014
mkdir
baileympearson Dec 4, 2024
a32a62f
no errexit
baileympearson Dec 4, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 1 addition & 3 deletions .github/workflows/build.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,4 @@
on:
pull_request:
branches: [main]
workflow_dispatch: {}
workflow_call: {}

Expand All @@ -10,7 +8,7 @@ jobs:
host_tests:
strategy:
matrix:
os: [macos-latest, windows-2019]
os: [macos-latest, windows-2019, macos-13]
fail-fast: false
runs-on: ${{ matrix.os }}
steps:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/test.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ jobs:
host_tests:
strategy:
matrix:
os: [macos-latest, windows-2019]
os: [macos-latest, windows-2019, macos-13]
node: [16.20.1, 18.x, 20.x, 22.x]
fail-fast: false
runs-on: ${{ matrix.os }}
Expand Down
2 changes: 1 addition & 1 deletion .release-please-manifest.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
{
".": "2.0.0-alpha.1"
}
}
13 changes: 13 additions & 0 deletions binding.gyp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,19 @@
]
},
}
],
['OS=="mac"', {
'xcode_settings': {
"OTHER_CFLAGS": [
"-arch x86_64",
"-arch arm64"
],
"OTHER_LDFLAGS": [
"-arch x86_64",
"-arch arm64"
]
}
}
]
],
'cflags!': [ '-fno-exceptions' ],
Expand Down
3 changes: 2 additions & 1 deletion etc/install-zstd.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
#!/bin/sh
set -o xtrace
set -o errexit

clean_deps() {
rm -rf deps
Expand All @@ -20,7 +21,7 @@ build_zstd() {

# CMAKE_RC_FLAGS is a workaround for a bug in 1.5.6 that breaks compilation on windows.
# The fix is merged but not yet released. see https://github.com/facebook/zstd/issues/3999
cmake -DCMAKE_RC_FLAGS="$(pwd)/lib" -DZSTD_MULTITHREAD_SUPPORT=OFF -DZSTD_BUILD_SHARED=OFF .
cmake -DCMAKE_RC_FLAGS="$(pwd)/lib" -DZSTD_MULTITHREAD_SUPPORT=OFF -DZSTD_BUILD_SHARED=OFF -DCMAKE_OSX_ARCHITECTURES=x86_64;arm64 .
cmake --build .
}

Expand Down
3 changes: 2 additions & 1 deletion package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,8 @@
"node": ">= 16.20.1"
},
"scripts": {
"install": "prebuild-install --runtime napi || node-gyp rebuild",
"install": "prebuild-install --runtime napi || npm run clean-install",
"clean-install": "npm run install-zstd && npm run compile",
"compile": "node-gyp rebuild",
"test": "mocha test/index.test.js",
"install-zstd": "bash etc/install-zstd.sh",
Expand Down
Loading