-
Notifications
You must be signed in to change notification settings - Fork 180
Duplicate, mismatched modules on npm #68
Comments
+1 just ran into this as well |
same |
This wreaks havoc on Windows/case insensitive filesystems when
Apparently my recommendation isn't possible due to changes in |
+1 |
👍 I can no longer install cucumber that depends on browserify. |
this is not a problem in linux environments, think @callum is right about case sensitivity. fails on osx here. |
Is there anything we can do in the meantime? It affects cordova and phonegap as well. |
i used a linux vm to install the dependencies |
Upgrading to the latest browserify solved it for me. Here is what they have done: browserify/browserify#1247. I guess each package that depends on JSONStream should do that dependency update. |
browserify@10.1.2 won't have this problem. it seems to only affect 5.0.0-10.1.1. But cross-publishing doesn't seem to be possible because JSONStream (w/ caps) isn't updatable npm/npm#7195 |
+1 |
3 similar comments
+1 |
+1 |
+1 |
+1. Updating browserify to 10.1.2 solved the issue for me |
Updating browserify works, but other dependencies like grunt-browserify are still effected. I tried doing an |
This has stopped elixir from installing as well as reported here http://laravel.io/forum/05-07-2015-elixir-wont-install?page=1 |
+1 |
1 similar comment
+1 |
JSONStream is a weird case because it has capitals which are no longer allowed but it seemed to have been grandfathered in, the JSONStream/jsonstream thing has been around for a while and previously didn't cause any issues |
+1 seeing this for installing the latest cordova, phonegap and Ionic (depends on Cordova) See reports here: |
I can't remove JSONStream, that would break thousands of modules. I had to publish jsonstream because npm broke my ability to update JSONStream npm/npm#7195 They have been saying they would fix it, so that legacy uppercase modules could still publish, but not new modules (this had been the behavior for a long time, until just recently...) I was then unable to publish new versions of JSONStream, however, pull requests where coming in, this this happened. so far there are only 8 things that depend on jsonstream (though, one of them is browserify, which everyone uses) |
Looking at npm/npm#7195, the validation is all done in npm and not on the server (as they were discussing adding server side validation). Based on that, would it be possible to install an older version of npm to push up an update to JSONStream to cross-publish? |
To @TheSharpieOne's point... The validation seems to happen here Though I don't know if it's right for @dominictarr to patch his npm so he can publish. |
Okay I published JSONStream at 1.0.2 and 1.0.3 does this still happen? |
@dominictarr IT WORKS!!!! |
thanks 👍 |
Thanks! |
Crisis [mostly] averted! |
Great! :) |
You get these warnings on OS X, but browserify works:
|
okay next step is just to get browserify to go back to JSONStream |
@dominictarr Ok... I'll switch browserify and the various deps over now |
I'm now getting shasum errors, cleaning cache for
Note this seems to happen only when using our internal registry (NPMe). Is there some action I need to take to update our version of jsonstream? |
@englercj If you can, upgrade to npmE 2.0.0, which has a fix for this issue (older versions of npmE just went ahead and lowercased all of the package names when the follower was replicating them, which is pretty obviously what's causing the problem here). The easiest way to upgrade the npmE distribution is:
If that doesn't get you sorted out, or is impossible, drop a line to support@npmjs.com and we'll get you sorted out. |
Is there a plan here either with npm or with other packages that use JSONStream as a dependency? I know Browserify updated quickly (thanks for that), but my repo has multiple other dependencies that rely on this package. I'm sure I'm not alone here. |
@dominictarr it looks like the conclusion to all this is that |
done. On Fri, Jun 19, 2015 at 7:44 PM, Ryan Graham notifications@github.com
|
Browserify looks for a specific version of JSONStream here https://github.com/substack/node-browserify/blob/master/package.json#L48
Is there a case sensitivity issue going on? This happened after upgrading node and npm. It fails on install because it tries to install JSONStream and not jsonstream where the version is less than specified by browserify.
The text was updated successfully, but these errors were encountered: