-
Notifications
You must be signed in to change notification settings - Fork 983
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
node-dtrace-provider should be an optionalDependency #100
Comments
Yep, this is the 3rd or 4th time this has come up. dtrace-provider is available on linux (I had patched it with a bunch of #ifdef's), but not windows. I will look into the optionalDependency trick and stubbing it out internally. |
…t (will require npm 1.1.5 to work smoothly)
To work smoothly with |
Resolving this. |
Isn't the latest version of npm 1.1.4? I tried npm update npm and got only 1.1.4. |
@mypark Yes. npm 1.1.5 will come out soon to address the shortcoming here. |
I just wanted to ask about this issue. I am seeing the following error message from restify ...
... using npm version 1.1.12, and restify version 1.3.0. It looks like I can continue using restify, despite that message. Is this what was changed with npm version 1.1.5? Did previous versions cause this to error out completely? Or, am I still seeing this error as it was originally reported for this issue? I just want to be sure to be clear what it is I am seeing. Thanks. |
Hi Ryan, So what changed was that as an optionalDependency, NPM won't blow up tl;dr, npm let you install, it's not really there, it whines on m On Sun, Mar 25, 2012 at 4:44 PM, Ryan Cole
|
@mcavage: I am seeing the same thing on heroku still...
|
Are you sure that error is actually from this? (that log makes it On Mon, Apr 16, 2012 at 8:13 PM, Chris Hoffman
|
@mcavage: you're probably right. Sorry. |
No worries. The [Error: ...] thing is from stderr, as opposed to it m On Tue, Apr 17, 2012 at 7:42 AM, Chris Hoffman
|
@mcavage : The problem was related to a file log, and arbitrary file writing is a no-no for Heroku. Thanks for pointing out the obvious, original problem |
I'm on windows. I don't have dtrace. I don't have python. All I want is to install restify but I can't. It says that I don't have python, therefore I can't compile because dtrace tries to compile with node-gyp... |
Reinstalling from http://nodejs.org/download/ helps. |
Restify is a great platform. It's a shame that it can't be used on Windows or Linux.
Possible workaround: port dtrace to Linux and Windows.
Also acceptable: list dtrace as an optional dependency, and don't use it if it's not available.
The text was updated successfully, but these errors were encountered: