-
Notifications
You must be signed in to change notification settings - Fork 30
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
Error: spawn ENOENT #48
Comments
If you want to reproduce, throw this in a
|
Seeing the same error here on OSX 10.11.3:
|
I don't remember the context with this but it make sense that this fails because the module is not installed yet and you are trying to run this in the PREINSTALL step. |
I'm not clear how it worked before, seems like there are some docs suggesting that it was possible before. |
@vladikoff What your'e saying makes sense, but it can't be the issue, because I manually installed the package with Because I mentioned a work-around, this case can be closed, but I suggest the README should be updated with the workaround mentioned. |
DISCLAIMER: I don't use Node, so I might be talking shit. @tylercollier Sounds like either If you don't know how, the output of |
@KellerFuchs should I gave a simple set of instructions for reproduction. One person said they saw the same problem, but @vladikoff didn't mention if he tried reproducing. |
The instructions in the README say to add this to
package.json
:However, when I do that, I get this error:
To fix, I changed it to be:
which seems to work fine.
I'm guessing this has something to do with my environment (Ubuntu 14.04, using node 4.2.3 with nodenv, using npm 3.5.1)? Or I must be missing something if no one else has reported this. But what's happening makes sense to me. Simply having
lockdown
as a preinstall script shouldn't work, unless you've addednode_modules/.bin
to your PATH (environment variable) or something, right? But that's not part of the README.The text was updated successfully, but these errors were encountered: