node-bin-gen 11.2.1 → 11.2.3

package/index.js

@@ -4,17 +4,18 @@
 import { open, unlink, mkdir, writeFile, readFile } from "node:fs/promises";
 import { Writable, Transform } from "node:stream";
 import { join, resolve, dirname } from "node:path";
-import { debuglog } from "node:util";
-import { execFile, spawn } from "node:child_process";
+import { debuglog, promisify } from "node:util";
+import { execFile as execFile_, spawn } from "node:child_process";
 import { fileURLToPath } from "url";
 
 import verr from "verror";
-import rimraf from "rimraf";
+import {rimraf} from "rimraf";
 import zlib from "zlib";
 import yargs from "yargs";
 
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = dirname(__filename);
+const execFile = promisify(execFile_);
 
 const { VError } = verr;
 
@@ -125,7 +126,7 @@ async function buildArchPackage(os, cpu, version, pre) {
     await res.body.pipeTo(Writable.toWeb(f.createWriteStream()));
     f.close();
 
-    const running = execFile("unzip", [
+    const running = await execFile("unzip", [
       "-d",
       `${dir}/bin`,
       "-o",
@@ -134,10 +135,8 @@ async function buildArchPackage(os, cpu, version, pre) {
       `${base}/node.exe`,
     ]);
 
-    await new Promise((y, n) => {
-      running.on("error", n);
-      running.on("exit", y);
-    });
+    if (running.stderr) console.warn("error from unzip", running.stderr);
+
     await unlink(filename);
   } else {
     const c = spawn("tar", ["--strip-components=1", "-C", dir, "-x"], {
@@ -147,8 +146,11 @@ async function buildArchPackage(os, cpu, version, pre) {
     const unzip = Transform.toWeb(zlib.createGunzip());
 
     await res.body.pipeThrough(unzip).pipeTo(Writable.toWeb(c.stdin));
+    c.stdin.end();
   }
 
+  debug("Finished unpacking into", dir);
+
   await writeFile(join(dir, "package.json"), JSON.stringify(pkg, null, 2));
   return pkg;
 }
@@ -200,6 +202,7 @@ async function main() {
       "linux-x64",
       "linux-x86",
       "sunos-x64",
+      "win-arm64",
       "win-x64",
       "win-x86",
     ];
@@ -212,7 +215,7 @@ async function main() {
       return (
         !/^headers|^src/.test(f) &&
         !/pkg$/.test(f) &&
-        !/^win-...-(exe|msi|7z)/.test(f)
+        !/^win-([^-]+)-(exe|msi|7z)/.test(f)
       );
     })
     .map(function (f) {
@@ -224,9 +227,9 @@ async function main() {
       };
     });
 
-  await Promise.all(
-    binaries.map((v) => buildArchPackage(v.os, v.cpu, version, pre))
-  );
+  for (const v of binaries) {
+    await buildArchPackage(v.os, v.cpu, version, pre)
+  }
 
   const pkg = await buildMetapackage(version + (pre != null ? "-" + pre : ""));
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "node-bin-gen",
-  "version": "11.2.1",
+  "version": "11.2.3",
   "description": "Generate a node binary package",
   "author": "Aria Stewart <aredridel@dinhe.net>",
   "main": "index.js",
@@ -15,7 +15,7 @@
     "node": ">=18.0.0"
   },
   "dependencies": {
-    "rimraf": "^4.0.4",
+    "rimraf": "^5.0.1",
     "verror": "^1.9.0",
     "yargs": "^17.0.1"
   },