Skip to content

Commit

Permalink
fix: lock adm-zip to version 0.5.12
Browse files Browse the repository at this point in the history
0.5.13+ is broken, extract empty on Windows

- cthackers/adm-zip#538
- fannheyward/coc-rust-analyzer#1281
  • Loading branch information
fannheyward committed Feb 21, 2025
1 parent e801a88 commit 1d892ba
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 8 deletions.
10 changes: 5 additions & 5 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@
"url": "https://github.com/clangd/node-clangd/issues"
},
"dependencies": {
"adm-zip": "^0.5.14",
"adm-zip": "0.5.12",
"node-fetch": "^2.7.0",
"readdirp": "^4.1.1",
"rimraf": "^6.0.1",
Expand Down
5 changes: 3 additions & 2 deletions src/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -323,8 +323,9 @@ namespace Install {
}
await ui.slow(
ui.localize('Extracting {0}', asset.name),
new Promise((resolve) => {
zip.extractAllToAsync(extractRoot, true, false, resolve);
new Promise<void>((resolve) => {
zip.extractAllTo(extractRoot, true, false);
resolve();
}),
);
const clangdPath = path.join(extractRoot, executable.entryName);
Expand Down

0 comments on commit 1d892ba

Please sign in to comment.