Skip to content
This repository has been archived by the owner on Jul 14, 2021. It is now read-only.

Update dependency links in NPM package #51

Closed
myndzi opened this issue Aug 30, 2013 · 3 comments
Closed

Update dependency links in NPM package #51

myndzi opened this issue Aug 30, 2013 · 3 comments

Comments

@myndzi
Copy link

myndzi commented Aug 30, 2013

The optional dependency in package.json pointing to https://github.com/godsflaw/nodeunit/tarball/master is no longer supported and causes npm installs to stall out horribly (and therefore possibly not run the rebuild script, which causes the module not to work at all)

This should be fixable simply by changing the url to https://github.com/godsflaw/nodeunit/archive/master.tar.gz

@godsflaw
Copy link
Collaborator

I fixed this earlier, are you seeing a new problem?

@myndzi
Copy link
Author

myndzi commented Aug 30, 2013

Don't know what counts as earlier, but I just did a clean "npm install" and still received the same error

@godsflaw
Copy link
Collaborator

Ahh, I forgot to publish it to npm

godsflaw added a commit to godsflaw/node-msgpack that referenced this issue Sep 24, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants