Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Installation Instructions #9

Closed
aackerman opened this issue Apr 16, 2012 · 7 comments
Closed

Installation Instructions #9

aackerman opened this issue Apr 16, 2012 · 7 comments

Comments

@aackerman
Copy link

I was able to figure it out but it would be great for others if you could add the instructions that the project needs to be built with node-gyp before using. Or if it's possible to setup autobuild when the project is downloaded from npm that would be even better. Thanks.

@wadey
Copy link
Owner

wadey commented Apr 16, 2012

Are you using an old version of NPM? The latest versions should build the project with gyp when installing.

@wadey
Copy link
Owner

wadey commented Apr 16, 2012

Example of installing with npm 1.1.15, showing the building is done:

$ npm install microtime
npm http GET https://registry.npmjs.org/microtime
npm http 200 https://registry.npmjs.org/microtime
npm http GET https://registry.npmjs.org/bindings
npm http 200 https://registry.npmjs.org/bindings

> [email protected] install /Users/wade/sandbox/node_modules/microtime
> node-gyp rebuild

info it worked if it ends with ok 
spawn python [ '/Users/wade/.node-gyp/0.6.14/tools/gyp_addon',
  'binding.gyp',
  '-I/Users/wade/sandbox/node_modules/microtime/build/config.gypi',
  '-f',
  'make' ]
spawn make [ 'BUILDTYPE=Release', '-C', 'build' ]
  CXX(target) Release/obj.target/microtime/src/microtime.o
  SOLINK_MODULE(target) Release/microtime.node
  SOLINK_MODULE(target) Release/microtime.node: Finished
info done ok 
[email protected] ./node_modules/microtime
└── [email protected]

@aackerman
Copy link
Author

Yep I was on 1.1.0, if it works with newer versions everything is good.

@wadey
Copy link
Owner

wadey commented Apr 16, 2012

I'll research what version is required and put a note in the README

@nategood
Copy link

👍

I too ran into this issue. A note in the README would be very helpful.

@evenwestvang
Copy link

+1

@wadey
Copy link
Owner

wadey commented Oct 9, 2012

Added note to README: 3f28c7e

@wadey wadey closed this as completed Oct 9, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants