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 peer-id 0.12.3 incompatible with other 0.12.x #96

Closed
npfoss opened this issue Jul 19, 2019 · 7 comments
Closed

npm package peer-id 0.12.3 incompatible with other 0.12.x #96

npfoss opened this issue Jul 19, 2019 · 7 comments

Comments

@npfoss
Copy link
Contributor

npfoss commented Jul 19, 2019

I just tried to update the ipfs npm package, and it bumped my version of peer-id from 0.12.2 to 0.12.3 (because many things in the ipfs tree rely on ~0.12.x or ^0.12.x), but then I started getting this error upon building with browserify:

Cannot find module 'babelify' from '/path/to/my/app/node_modules/ipfs/node_modules/peer-id/node_modules/async'

Could you please remove version 0.12.3 or replace it with something compatible with all the other packages asking for ~0.12.0?

Update:

Steps to reproduce the error:

package.json

{
  "scripts": {
    "build": "browserify index.js -o app.js -d"
  },
  "dependencies": {
    "browserify": "^16.3.0",
    "ipfs": "^0.36.4"
  }
}

index.js

require('ipfs');

run

npm install
npm run build

Also available to be cloned here: https://github.com/npfoss/js-ipfs-bug-code

@npfoss npfoss changed the title peer-id 0.12.3 incompatible with other 0.12.x npm package peer-id 0.12.3 incompatible with other 0.12.x Jul 19, 2019
@npfoss
Copy link
Contributor Author

npfoss commented Jul 21, 2019

@jacobheun It looks like you have the power to fix this, could you please take a look?

@alanshaw
Copy link
Member

This looks like it might be an issue with v3 of async caolan/async#1653 (comment)

@npfoss
Copy link
Contributor Author

npfoss commented Jul 22, 2019

It totally was. Thank you! Sorry for the misdiagnosis @ peer-id team

@npfoss npfoss closed this as completed Jul 22, 2019
@jacobheun
Copy link
Contributor

I think I am going to go ahead and cut a new patch release to roll back to async 2.6. Since we're migrating to await we wont be using the package much longer and right now it's just going to add some bloat to the bundle size. I am just doing some local testing and then will look to release the patch in the 0.12 line.

@alanshaw
Copy link
Member

@jacobheun multistream-select is also using async@3

@jacobheun
Copy link
Contributor

@alanshaw yeah, noticed that last night when I was testing, I will roll the version back there as well.

jacobheun added a commit to multiformats/js-multistream-select that referenced this issue Jul 23, 2019
jacobheun added a commit to multiformats/js-multistream-select that referenced this issue Jul 23, 2019
@jacobheun
Copy link
Contributor

Just closing the loop here, both peer-id (12.x) and multistream-select have had their async deps reverted

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