Skip to content
This repository
Browse code

build: set `process.platform` to "sunos" on SunOS

gyp sets it to "solaris" by default, but versions of node v0.6.x and
older would report "sunos". Let's keep things consistent.

Fixes #3944.
  • Loading branch information...
commit 606a9339891a68144c6047356480f199ce04ae4c 1 parent 4784ea1
Nathan Rajlich authored August 29, 2012

Showing 1 changed file with 8 additions and 0 deletions. Show diff stats Hide diff stats

  1. 8  node.gyp
8  node.gyp
@@ -227,6 +227,14 @@
227 227
             '-lkstat',
228 228
             '-lumem',
229 229
           ],
  230
+          'defines!': [
  231
+            'PLATFORM="solaris"',
  232
+          ],
  233
+          'defines': [
  234
+            # we need to use node's preferred "sunos"
  235
+            # rather than gyp's preferred "solaris"
  236
+            'PLATFORM="sunos"',
  237
+          ],
230 238
         }],
231 239
       ],
232 240
       'msvs-settings': {

0 notes on commit 606a933

Please sign in to comment.
Something went wrong with that request. Please try again.