Skip to content
This repository

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse code

build: compile with -D__DARWIN_64_BIT_INO_T on OS X

Fixes a struct stat size mismatch on 64 bits machines that made Node crash with
a EXC_BAD_ACCESS on startup.

Fixes #2061. Solution proposed by Paddy Byers.
  • Loading branch information...
commit 8bed0a33fe8a2146aad4c72f5390687fe77cd96d 1 parent 1e34c85
Ben Noordhuis authored November 18, 2011

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

  1. 5  wscript
5  wscript
@@ -461,6 +461,11 @@ def configure(conf):
461 461
   conf.env.append_value('CPPFLAGS',  '-D_LARGEFILE_SOURCE')
462 462
   conf.env.append_value('CPPFLAGS',  '-D_FILE_OFFSET_BITS=64')
463 463
 
  464
+  # Apparently _LARGEFILE_SOURCE and _FILE_OFFSET_BITS isn't always enough
  465
+  # on OS X, see https://github.com/joyent/node/issues/2061 for details.
  466
+  if sys.platform.startswith('darwin'):
  467
+    conf.env.append_value('CPPFLAGS', '-D__DARWIN_64_BIT_INO_T=1')
  468
+
464 469
   # Makes select on windows support more than 64 FDs
465 470
   if sys.platform.startswith("win32"):
466 471
     conf.env.append_value('CPPFLAGS', '-DFD_SETSIZE=1024');

0 notes on commit 8bed0a3

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