Skip to content
This repository has been archived by the owner on Feb 18, 2024. It is now read-only.

Delete [email protected] #784

Merged
merged 8 commits into from
Feb 28, 2016
Merged

Delete [email protected] #784

merged 8 commits into from
Feb 28, 2016

Conversation

salfield
Copy link
Contributor

This is causing an old and broken version of has-cors to get installed when you try and install socket.io.

Without it I am able to install socket.io, pending a proper fix to:

jspm/jspm-cli#1571

This is causing an old and broken version of has-cors to get installed when you try and install socket.io.
"parseqs": "0.0.2",
"component-inherit": "0.0.3"
}
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rather than deleting, we should create a new override for the later version that this isn't needed on.

@salfield
Copy link
Contributor Author

Sure, that makes sense. However, the has-cors dependency can't actually be installed at 1.0.3 due to the use of a git archive as a dependency - so I can't see how this override will work in any case:

component/has-cors#1
component/has-cors#4

Don't use the old version of "accepts", which requires a version of "negotiator" using dynamic requires.
Allow socket.io to import the socket.io-client package.json.
This doesn't need the json loader - it can just be loaded as javascript.
It seems that I must override all dependencies if I override 1. Which is a shame because I only want to override "accepts".
utf-8-validate and bufferutil are conditionally required causing this module to fail.
@salfield
Copy link
Contributor Author

This now contains everything needed to get socket.io installed and running. I've re-added the (imo broken) version of engine.io-client and added an empty one which allows the latest version to install. I'm not clear why a requirement for [email protected] would be affected by an override for 1.4.5 in any case?

Socket.io still cannot run using the npm version due to three issues:

  1. URL encoding of version string in package-name.js jspm-cli#1571 - temporarily fixed without any obvious adverse effects in my fork of jspm
  2. Superfluous require('socket.io-client') in index.js socketio/socket.io#2463
  3. require.resolve support npm#88

Issues 2&3 are fixed in: salfield/socket.io

@guybedford
Copy link
Member

Can you remind me if you're running jspm 0.16 or jspm 0.17 here?

guybedford added a commit that referenced this pull request Feb 28, 2016
@guybedford guybedford merged commit 33721bc into jspm:master Feb 28, 2016
@salfield
Copy link
Contributor Author

0.17.0-beta.9

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

Successfully merging this pull request may close these issues.

2 participants