-
Notifications
You must be signed in to change notification settings - Fork 30.2k
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
NPM git dependency's not installing #295
Comments
/cc @othiym23 |
It looks like the url parsing is a little off. With node v0.10.35, the
but with the latest nightly:
Looks like it may have been introduced in c90eac7 |
@evanlucas if you're sure that's the commit could you cc the author into this thread? |
/cc @jondavidjohn |
I tried rolling back that commit and building but still have the same issue |
ah, sorry, looks like 6120472 was the original |
After looking further into it, it looks like d312b6d#diff-bbc5176adff1bbc01acd61bfc85d049fR407 made a change that will also prevent npm from installing git dependencies as prior to that commit, |
#299 has landed but |
This reverts commit d312b6d. Reverted for breaking `npm install` with git+ssh URLs. Conflicts: doc/api/url.markdown Fixes: nodejs#295 PR-URL: nodejs#303 Reviewed-By: Rod Vagg <[email protected]>
This should be fixed by commits a1e54d6 and 913addb. Thanks for the bug report, @corbinu, and thanks @evanlucas for helping out. |
Thanks so much guys. Working for me. Will keep testing. Can't wait for the release |
i've ever meet this issue |
Not sure if this is an NPM issue, but after installing the io.js nightly I am unable to install git dependancies. If this is an NPM issue I can log it there.
I was previously using
NPM 2.2.0
Node 0.12
On OS X 10.10
For example using this package.json
gives this error
The text was updated successfully, but these errors were encountered: