hughsk / disc

:chart_with_upwards_trend: Visualise the module tree of browserify project bundles and track down bloat.
http://hughsk.io/disc
Other
1.33k stars 83 forks source link

Outdated dependency `bl` causing install failure #46

Open kumavis opened 9 years ago

kumavis commented 9 years ago

here's my install failure on v0.12

╭─kumavis@xyzs-MacBook-Pro  ~/dev/dapp-sandbox ‹node-v0.12.7›  (master*) 
╰─$ npm i -g disc
/Users/kumavis/.nvm/versions/node/v0.12.7/bin/discify -> /Users/kumavis/.nvm/versions/node/v0.12.7/lib/node_modules/disc/bin/discify
/Users/kumavis/.nvm/versions/node/v0.12.7/lib
└─┬ disc@1.3.2 
  ├─┬ bl@0.7.0 
  │ └─┬ readable-stream@1.0.33 
  │   ├── core-util-is@1.0.1 
  │   ├── inherits@2.0.1 
  │   ├── isarray@0.0.1 
  │   └── string_decoder@0.10.31 
  ├─┬ browser-unpack@0.2.3 
  │ ├─┬ concat-stream@1.2.1 
  │ │ └─┬ bops@0.0.6 
  │ │   ├── base64-js@0.0.2 
  │ │   └── to-utf8@0.0.1 
  │ ├── esprima-fb@3001.1.0-dev-harmony-fb 
  │ └── minimist@0.0.5 
  ├── builtins@0.0.3 
  ├── commondir@0.0.1 
  ├── d3@3.5.6 
  ├── duplexer@0.1.1 
  ├─┬ file-tree@1.0.0 
  │ ├── async-reduce@0.0.1 
  │ └── flat@1.0.0 
  ├── flatten@0.0.1 
  ├── map-async@0.1.1 
  ├── opener@1.4.1 
  ├─┬ optimist@0.6.1 
  │ └── wordwrap@0.0.3 
  ├── plucker@0.0.0 
  ├── UNMET PEER DEPENDENCY stream-browserify@*
  ├── through@2.3.8 
  └── uniq@1.0.1 

npm WARN EPEERINVALID bl@0.7.0 requires a peer of stream-browserify@* but none was installed.
kumavis commented 9 years ago
╭─kumavis@xyzs-MacBook-Pro  ~/dev/dapp-sandbox ‹node-v0.12.7›  (master*) 
╰─$ npm -v                                                                                                                                                                                 127 ↵
3.3.3
kumavis commented 9 years ago

works on node 4

╭─kumavis@xyzs-MacBook-Pro  ~/dev/dapp-sandbox ‹node-v0.12.7›  (master*) 
╰─$ nvm use 4      
Now using node v4.0.0 (npm v2.14.2)
╭─kumavis@xyzs-MacBook-Pro  ~/dev/dapp-sandbox ‹node-v4.0.0›  (master*) 
╰─$ npm i -g disc
npm WARN peerDependencies The peer dependency stream-browserify@* included from bl will no
npm WARN peerDependencies longer be automatically installed to fulfill the peerDependency 
npm WARN peerDependencies in npm 3+. Your application will need to depend on it explicitly.
/Users/kumavis/.nvm/versions/node/v4.0.0/bin/discify -> /Users/kumavis/.nvm/versions/node/v4.0.0/lib/node_modules/disc/bin/discify
disc@1.3.2 /Users/kumavis/.nvm/versions/node/v4.0.0/lib/node_modules/disc
├── builtins@0.0.3
├── flatten@0.0.1
├── opener@1.4.1
├── commondir@0.0.1
├── plucker@0.0.0
├── uniq@1.0.1
├── map-async@0.1.1
├── duplexer@0.1.1
├── through@2.3.8
├── file-tree@1.0.0 (async-reduce@0.0.1, flat@1.0.0)
├── optimist@0.6.1 (wordwrap@0.0.3, minimist@0.0.10)
├── d3@3.5.6
├── browser-unpack@0.2.3 (minimist@0.0.5, esprima-fb@3001.1.0-dev-harmony-fb, concat-stream@1.2.1)
├── stream-browserify@2.0.1 (inherits@2.0.1, readable-stream@2.0.4)
└── bl@0.7.0 (readable-stream@1.0.33)
kumavis commented 9 years ago

and theres the rub:

npm WARN peerDependencies The peer dependency stream-browserify@* included from bl will no
npm WARN peerDependencies longer be automatically installed to fulfill the peerDependency 
npm WARN peerDependencies in npm 3+. Your application will need to depend on it explicitly.
jasonkarns commented 8 years ago

This isn't an issue with disc, it's an issue with bl. bl 0.7.0 lists a peerDep of stream-browserify. Latest version of bl (1.0.0) does not have stream-browserify as a peerDep. Indeed, 0.8.0 removed the peerDep. So disc could avoid this warning message by bumping bl to 0.8.0 or newer.

kumavis commented 8 years ago

@jasonkarns thanks for the clarification

TehShrike commented 8 years ago

Just ran into this now. A merge would be great!

Elyx0 commented 8 years ago

Ran into this too while installing gulp-uglify. Dang.

revelt commented 7 years ago

Same problem here, I even installed stream-browserifyglobally, separately, nothing helps.

stream_browserify