Add prebuilds for Windows and C Client v3.8.2 (#15)

* fix(windows): build with extra CMAKE flags * fix(windows): apply patches is still needed * feat(windows): add prebuild for node 18
DavidVujic · Aug 7, 2023 · 193d592 · 193d592
1 parent 14e47df
commit 193d592
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 1 deletion.
diff --git a/prebuilds/win32-x64/node.abi108.node b/prebuilds/win32-x64/node.abi108.node
diff --git a/scripts/build.js b/scripts/build.js
@@ -16,7 +16,7 @@ shell.cd(`${env.sourceFolder}`);
 
 if (env.isWindows) {
     const output = env.isVerbose ? '' : ' > NUL';
-    exec(`cmake -DWANT_SYNCAPI=OFF -DCMAKE_GENERATOR_PLATFORM=${process.arch} .${output}`);
+    exec(`cmake -DWANT_SYNCAPI=OFF -DCMAKE_GENERATOR_PLATFORM=${process.arch} .${output} -DWITH_OPENSSL=OFF -DWITH_CYRUS_SASL=OFF`);
     exec(`cmake --build .${output}`);
 } else {
     const flags = '-w';

diff --git a/scripts/prepublish.js b/scripts/prepublish.js
@@ -11,6 +11,15 @@ function clearPath() {
     shell.rm('-rf', env.sourceFolder);
 }
 
+function applyPatches() {
+    if (env.isWindows) {
+        const destination = `${env.sourceFolder}/src`;
+        shell.sed('-i', '#include "zookeeper_log.h"', '#include "zookeeper_log.h"\n#include "winport.h"\n', `${destination}/zk_log.c`);
+        shell.sed('-i', '#include "zookeeper.h"', '#include "winport.h"\n#include "zookeeper.h"\n', `${destination}/zk_adaptor.h`);
+        shell.sed('-i', '#include "zk_adaptor.h"', '#include "zk_adaptor.h"\n#include "winport.h"\n', `${destination}/zookeeper.c`);
+    }
+}
+
 if (env.isAlreadyBuilt) {
     shell.echo('Zookeeper has already been built');
     shell.exit(0);
@@ -31,6 +40,7 @@ decompress(env.downloadedFileName, './', {
     ],
 }).then(() => {
     shell.echo('Decompressed file');
+    applyPatches();
 }).catch((e) => {
     shell.echo(`Error: ${e.message}`);
     shell.exit(1);