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

Use of peerDependencies for vow breaks shrinkwrapping of packages using this lib #10

Closed
cvrebert opened this issue May 8, 2014 · 16 comments

Comments

@cvrebert
Copy link

cvrebert commented May 8, 2014

$ cat package.json  # Note: csscomb depends on vow-fs (& thus vow-queue)
{
  "name": "foobar",
  "private": true,
  "devDependencies": {
    "csscomb": "^2.0.4"
  }
}
$ npm install
<succeeds>
$ npm ls
foobar@ /Users/chris.rebert/code/foobar
└─┬ [email protected]
  ├── [email protected]
  ├── [email protected]
  ├─┬ [email protected]
  │ ├── [email protected]
  │ └── [email protected]
  ├── [email protected]
  └─┬ [email protected]
    ├── [email protected]
    └─┬ [email protected]
      └── UNMET DEPENDENCY vow ~0.3.9

npm ERR! missing: vow@~0.3.9, required by [email protected]
npm ERR! not ok code 0
$ npm shrinkwrap
npm ERR! Error: Problems were encountered
npm ERR! Please correct and try again.
npm ERR! missing: vow@~0.3.9, required by [email protected]
npm ERR!     at shrinkwrap_ (/usr/local/share/npm/lib/node_modules/npm/lib/shrinkwrap.js:30:15)
npm ERR!     at /usr/local/share/npm/lib/node_modules/npm/lib/shrinkwrap.js:24:5
npm ERR!     at /usr/local/share/npm/lib/node_modules/npm/lib/ls.js:44:30
npm ERR!     at /usr/local/share/npm/lib/node_modules/npm/node_modules/read-installed/read-installed.js:130:5
npm ERR!     at /usr/local/share/npm/lib/node_modules/npm/node_modules/read-installed/read-installed.js:258:14
npm ERR!     at cb (/usr/local/share/npm/lib/node_modules/npm/node_modules/slide/lib/async-map.js:48:11)
npm ERR!     at /usr/local/share/npm/lib/node_modules/npm/node_modules/read-installed/read-installed.js:258:14
npm ERR!     at cb (/usr/local/share/npm/lib/node_modules/npm/node_modules/slide/lib/async-map.js:48:11)
npm ERR!     at /usr/local/share/npm/lib/node_modules/npm/node_modules/read-installed/read-installed.js:258:14
npm ERR!     at cb (/usr/local/share/npm/lib/node_modules/npm/node_modules/slide/lib/async-map.js:48:11)
npm ERR! If you need help, you may report this *entire* log,
npm ERR! including the npm and node versions, at:
npm ERR!     <http://github.com/npm/npm/issues>

npm ERR! System Darwin 13.1.0
npm ERR! command "node" "/usr/local/bin/npm" "shrinkwrap"
npm ERR! cwd /Users/chris.rebert/code/foobar
npm ERR! node -v v0.10.26
npm ERR! npm -v 1.4.9
npm ERR! 
npm ERR! Additional logging details can be found in:
npm ERR!     /Users/chris.rebert/code/foobar/npm-debug.log
npm ERR! not ok code 0
@dfilatov
Copy link
Owner

dfilatov commented May 8, 2014

I don't understand why "missing: vow@~0.3.9, required by [email protected]".
[email protected] which was installed earlier, must satisfy that condition.

@cvrebert
Copy link
Author

cvrebert commented May 8, 2014

I think because vow-fs and vow-queue are at different levels in the tree, thus they don't share siblings?

Also, the npm folks are debating deprecating the peerDependencies feature: npm/npm#5080

@XhmikosR
Copy link

Ping

@pwmckenna
Copy link

👍

@vladikoff
Copy link

Why is this issue happening? Was this a breaking change in NPM?

@cvrebert
Copy link
Author

Sort of. Until recently, npm didn't error out in such situations; now it does.

@josegonzalez
Copy link
Contributor

Well this is still broken, does anyone know of a workaround? It totally breaks listing the npm cache as well, which does wonders for my sanity.

@pwmckenna
Copy link

you can just downgrade npm to 1.4.0 or so. not ideal, but saved me big time.

@vladikoff
Copy link

I had to downgrade to 1.3
On May 30, 2014 2:08 PM, "Patrick Williams" [email protected]
wrote:

you can just downgrade npm to 1.4.0 or so. not ideal, but saved me big
time.


Reply to this email directly or view it on GitHub
#10 (comment).

@cvrebert
Copy link
Author

cvrebert commented Jun 5, 2014

@dfilatov So, any chance of either removing peerDependencies from vow or commenting on npm/npm#5080 ?

@dfilatov
Copy link
Owner

dfilatov commented Jun 6, 2014

@cvrebert I've released vow-queue 0.3.1 without peerDependencies from vow. Is it enough?

@cvrebert
Copy link
Author

cvrebert commented Jun 6, 2014

@dfilatov That helps, but:

@XhmikosR
Copy link

XhmikosR commented Jun 6, 2014

For csscomb I guess we'll have to bug the csscomb guys after the vow(*) stuff is fixed.

@tonyganch
Copy link

Hi, guys.
Let's discuss all CSScomb related stuff here: csscomb/csscomb.js#229

@dfilatov
Copy link
Owner

dfilatov commented Jun 9, 2014

@cvrebert I've released [email protected] which has no more peerDependencies from vow.

@dfilatov dfilatov closed this as completed Jun 9, 2014
@cvrebert
Copy link
Author

cvrebert commented Jun 9, 2014

Thanks again!

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

7 participants