lloyd / node-toobusy

Build Node.JS servers that don't fall over.
1.3k stars 128 forks source link

Compilation error on Joyent (SunOS) #12

Open cdarne opened 11 years ago

cdarne commented 11 years ago

Hi,

I'm having a hard time trying to install the module. Here's the build error I get:

CXX(target) Release/obj.target/toobusy/toobusy.o
cc1plus: erreur: option "-fno-tree-vrp" de la ligne de commande non reconnue

The server runs SunOS 5.11 and GCC 3.4.6. Any idea? Is it safe to remove this option?

lloyd commented 11 years ago

it looks like this is an upstream bug? https://github.com/joyent/node/issues/3863

lloyd commented 11 years ago

I believe it's safe to remove this flag for gcc 3.4.6, because I believe it's disabling an optimization that did not exist in that compiler version. I'd accept a patch to node-toobusy which made it unnecessary to hack out this compiler flag, but it's not apparent to me exactly how to craft that patch :/ hth?