1722331758

Author: Oleh Kulykov

.github/workflows/nodejs-multi-platform.yml

@@ -8,10 +8,11 @@ on:
 
 jobs:
   build:
-    runs-on: ubuntu-latest
+    runs-on: ${{ matrix.os }}
 
     strategy:
       matrix:
+        os: [macos-latest, ubuntu-latest, windows-latest]
         node-version: ['13.x', '16.x', '20.x'] # min, dev, latest
 
     steps:

binding.gyp

@@ -6,7 +6,8 @@
         'LIBPLZMA_VERSION_BUILD=858',
         'LIBPLZMA_SHARED=1',
         'LIBPLZMA_NO_C_BINDINGS=1',
-        'LIBPLZMA_HAVE_STD=1'
+        'LIBPLZMA_HAVE_STD=1',
+        'LIBPLZMA_NO_CPP_RTTI=1'
       ],
       'sources': [
         'node/module.cpp',

src/plzma.cpp

@@ -36,16 +36,16 @@
 #include <time.h>
 
 #if defined(LIBPLZMA_MSC)
-#include <malloc.h>
+#  include <malloc.h>
 #endif
 
 #if defined(BUILDING_NODE_EXTENSION)
-#include <node.h>
-#include <uv.h>
+#  include <node.h>
+#  include <uv.h>
 #endif
 
 #ifndef RAND_MAX
-#define RAND_MAX 0x7fffffff
+#  define RAND_MAX 0x7fffffff
 #endif
 
 #if __has_feature(cxx_rtti)
@@ -65,7 +65,9 @@
 #  define RTTI_ENABLED 1
 #endif
 
-#if defined(RTTI_ENABLED) && defined(LIBPLZMA_NO_CPP_RTTI)
+#if !defined(RTTI_ENABLED) && !defined(LIBPLZMA_NO_CPP_RTTI)
+#  define LIBPLZMA_NO_CPP_RTTI 1
+#elif defined(RTTI_ENABLED) && defined(LIBPLZMA_NO_CPP_RTTI)
 #  error "CMake rtti configuration error."
 #endif