Skip to content

Commit 3610e77

Browse files
committed
fix: simplify Windows prebuild process by using run-prebuildify script
1 parent c52f048 commit 3610e77

File tree

2 files changed

+4
-19
lines changed

2 files changed

+4
-19
lines changed

.claude/settings.local.json

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,9 @@
55
"Bash(node:*)",
66
"Bash(npm run rebuild:*)",
77
"Bash(grep:*)",
8-
"Bash(npm run clean:*)"
8+
"Bash(npm run clean:*)",
9+
"Bash(gh run:*)"
910
],
1011
"deny": []
1112
}
12-
}
13+
}

.github/workflows/build.yml

Lines changed: 1 addition & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -182,23 +182,7 @@ jobs:
182182
183183
- name: Generate prebuilds (Windows)
184184
if: matrix.os == 'windows-latest'
185-
shell: pwsh
186-
run: |
187-
# Use full path to node-gyp to avoid spawn issues
188-
$env:Path = "$env:Path;$pwd\node_modules\.bin"
189-
190-
# Call node-gyp directly for each target
191-
node-gyp rebuild --target=$(node -p "process.versions.napi") --release
192-
193-
# Create prebuilds directory structure
194-
$platform = "win32-x64"
195-
$prebuildDir = "prebuilds\$platform"
196-
New-Item -ItemType Directory -Force -Path $prebuildDir | Out-Null
197-
198-
# Copy the built .node file
199-
Copy-Item "build\Release\hamlib.node" "$prebuildDir\node.napi.node"
200-
201-
Write-Host "Successfully created prebuild for $platform"
185+
run: node scripts/run-prebuildify.js
202186

203187
- name: Bundle dependencies (Windows)
204188
if: matrix.os == 'windows-latest'

0 commit comments

Comments
 (0)