search

TooTallNate / ref

Turn Buffer instances into "pointers"
http://tootallnate.github.com/ref
453 stars 141 forks source link

ref doesn't run node-gyp on npm install when being external dependency (of ref-struct) #40

Closed cookiengineer closed 8 years ago

cookiengineer commented 8 years ago

Hey,

I was installing ref-struct as a module, but installing ref-struct via npm install ref-struct will not build the proper bindings for ref inside node_modules/ref-struct/node_modules/ref then. I had to manually run node-gyp configure and node-gyp build afterwards within that folder.

How to fix that?

TooTallNate commented 8 years ago

That definitely shouldn't be the case. npm builds all native dependencies as it installs them, if not than it's an npm bug. What node/npm versions are you using?

cookiengineer commented 8 years ago

The version installed and downloaded around an hour ago. I frequently removed /usr/lib/node_modules/* to be sure it's up-to-date and set all dependency versions to * and removed ./node_modules before updating them.

I commited the stuff to github here so you can take a look at the dependencies. I also uploaded the node_modules folder where ./node_modules/ref-struct/node_modules/ref was not automatically built (so you need to execute node-gyp manually as described before).

In the npm install process there were no errors occuring despite all the tyical C++ warnings. I noticed that some ref- modules were requiring sudo, others were not - and I had no clue why.

TooTallNate commented 8 years ago

sudo should definitely never be required.

But if npm isn't compiling the dependencies as it installs them, then that sounds like an npm bug to me (or something strange with your setup). Perhaps try reporting at https://github.com/npm/npm because there's nothing I can do in this repo to fix it for you.