Skip to content
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 package broken #28

Closed
jeremylevy opened this issue May 13, 2015 · 9 comments · Fixed by #29
Closed

Npm package broken #28

jeremylevy opened this issue May 13, 2015 · 9 comments · Fixed by #29
Assignees
Labels

Comments

@jeremylevy
Copy link

The NPM package seems to be broken :

npm install hideShowPassword
fetch failed https://registry.npmjs.org/hideShowPassword/-/hideShowPassword-2.0.4.tgz
npm ERR! Error: 404 Not Found

By the way, thanks for the good work. It works very well.

@tylersticka
Copy link
Member

Hrm, I'm encountering the same error. Thanks for letting us know, we'll investigate...

@tylersticka
Copy link
Member

Huh, it worked for me a moment ago, but for @erikjung it did not.

According to NPM's Troubleshooting page:

  • It's most likely a temporary npm registry glitch. Check npm server status and try again later.
  • If the error persists, perhaps the published package is corrupt. Contact the package owner and have them publish a new version of the package.

But the status is currently green. The fact that it installs fine some of the time (and if I do an npm install ~/my/local/folder/hideShowPassword, that works fine, too) suggests that corruption is unlikely.

Some pages online say you need to set the npm registry, but running npm config get registry for me outputs the correct registry URL (https://registry.npmjs.org/).

Most perplexing, one of our other npm modules doesn't seem to have this problem at all...

Still investigating....... 💭

@jeremylevy
Copy link
Author

It worked for me BEFORE my npm cache was cleaned with:

npm cache clean

Once cleaned, the npm install has broken.

Thanks for the support !

@tylersticka
Copy link
Member

Yeah, that seems to be the case for me as well... cleaning the cache makes it break again.

@lyzadanger pointed out that it's perplexing to see zero downloads on the npm page, compared to another package which has many downloads despite being far less popular (it has about 1% of the amount of GitHub stars). Something's not right there!

@tylersticka tylersticka self-assigned this May 13, 2015
@tylersticka
Copy link
Member

I'm comparing the two package.json files and noticing a discrepancy in the way the repository.url field is specified.

For Leveller, it's defined as https://github.com/cloudfour/leveller.git.

For hideShowPassword, it's defined as git://github.com/cloudfour/hideShowPassword.git.

The NPM docs show examples that use the former (https), not the latter (git).

I'm going to modify the package.json and push a new version to see if that remedies the issue or not.

@tylersticka
Copy link
Member

Did not mean to close this.

So, I implemented the package.json change, but then there's a new wrinkle... I found out that npm doesn't allow publishing of packages with capital letters in their names. I'm not sure how we got a registry page at all before, but apparently that was a fluke or an oversight or a hiccup or something.

Still developing...

@tylersticka tylersticka reopened this May 13, 2015
tylersticka added a commit that referenced this issue May 13, 2015
@tylersticka
Copy link
Member

@jeremylevy What a long, strange trip it's been! Thanks for bearing with us.

Can you try npm install hideshowpassword (all lower-case) and see if that resolves your issue?

@jeremylevy
Copy link
Author

It works ! You rock ! 👍

@tylersticka
Copy link
Member

@jeremylevy Great, thanks so much for reporting this issue! It's embarrassing to think how long this npm package may have been broken. 😓

I spoke with the folks at NPM and they recommended deprecating hideShowPassword. So now, if you attempt to install the camel-case version, you should see a warning that advises you to use the lowercase version instead:

$ npm install hideShowPassword
npm WARN deprecated hideShowPassword@2.0.4: use lowercase version: hideshowpassword
...

Closing this issue now!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants