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

dist-tags parsed wrong? #55

Closed
aikomastboom opened this issue Mar 13, 2014 · 4 comments
Closed

dist-tags parsed wrong? #55

aikomastboom opened this issue Mar 13, 2014 · 4 comments

Comments

@aikomastboom
Copy link

Hi,

http://registry.npmjs.org/minijasminenode/

has:

"dist-tags": {
  "latest": "0.2.7",
  "canary": "0.3.0"
}

when asking sinopia to proxy:

uplinks:
  npmjs:
    url: https://registry.npmjs.org/

sinopia returns:

"dist-tags": {
"latest": "0.3.0",
"canary": "0.3.0"
}

In my case braking: https://github.com/angular/protractor/blob/master/package.json

  "minijasminenode": ">=0.2.7"
@aikomastboom
Copy link
Author

cross-reference: angular/protractor#611

@rlidwka
Copy link
Owner

rlidwka commented Mar 13, 2014

latest tag is intentionally ignored to avoid npm bugs (see #50)

although I'm thinking about changing it back recently

@yannickcr
Copy link
Contributor

We also have some problems related to this issue but with unpublished versions.

For example we are currently using the simple-xmpp package that have 0.1.18 as the latest version but also have an unpublished one in 0.1.91.

npm:

"dist-tags": {
  "latest": "0.1.18"
}

Sinopia:

"dist-tags": {
  "latest": "0.1.91"
}

Typing npm install simple-xmpp with Sinopia will install the 0.1.91, so the unpublished and wrong one.

Since the original npm bug is now fixed, is would be nice to stop ignoring the the latest tag to avoid those kind of problems.

@rlidwka
Copy link
Owner

rlidwka commented Apr 1, 2014

Since the original npm bug is now fixed, is would be nice to stop ignoring the the latest tag to avoid those kind of problems.

Fixed in [email protected], now "latest" tag is treated like all other tags.

closing as duplicate of #50

@rlidwka rlidwka closed this as completed Apr 1, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants