-
Notifications
You must be signed in to change notification settings - Fork 518
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
Installing fails when there is a space in the path #58
Comments
Ugh. Unfortunate side-effect of adding dtrace-provider as a dep. Thanks for the bug report. I'll see if I can get a patch to dtrace-provider, else I'll just log a bug there. :) |
Same issue as kelektiv/node.bcrypt.js#121 |
Here is the real issue referred to in that bug: nodejs/node-gyp#65 And the related node-dtrace-provider bug: chrisa/node-dtrace-provider#25 Closing the bunyan issue -- thought bunyan could mitigate by making dtrace-provider an optional dep, I'd rather not. |
Added nodejs/node-gyp#164 that should fix this. That'll require a new core node release with the new node-gyp tho. |
Thanks Trent! In the meantime, I'll just add a note to our README. Making the dtrace-provider an optional dependency is probably not necessary. |
When doing npm install -d on a path that contains a space, the following error is thrown:
The error goes away when removing the space from the path.
The text was updated successfully, but these errors were encountered: