Only build (and link lpcap) for mac

Use updated NAN expose for Electron compatibility
Bartel-C8 · Nov 10, 2023 · 12e1ce4 · 12e1ce4
1 parent a637ab6
commit 12e1ce4
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 5 deletions.
diff --git a/binding.gyp b/binding.gyp
@@ -6,11 +6,15 @@
       "include_dirs": [
         "<!(node -e \"require('nan')\")"
       ],
-      "link_settings": {
-          "libraries": [
+      "conditions": [
+        ['OS == "mac"', {
+          "link_settings": {
+            "libraries": [
               "-lpcap"
-          ]
-      }
+            ]
+          }
+        }]
+      ]
     }
   ]
 }
diff --git a/package.json b/package.json
@@ -13,6 +13,9 @@
     "type": "git",
     "url": "git://github.com/plus100kt/node_pcap.git"
   },
+  "os": [
+    "darwin"
+  ],
   "engines": {
     "node": ">=18.0.0"
   },

diff --git a/pcap_binding.cc b/pcap_binding.cc
@@ -147,5 +147,9 @@ void Initialize(Local<Object> exports)
 }
 
 DISABLE_WCAST_FUNCTION_TYPE
-NODE_MODULE(pcap_binding, Initialize)
+#if NODE_MAJOR_VERSION >= 10
+  NAN_MODULE_WORKER_ENABLED(pcap_binding, Initialize)
+#else
+  NODE_MODULE(pcap_binding, Initialize)
+#endif
 DISABLE_WCAST_FUNCTION_TYPE_END