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 shrinkwrap produces error with hapi #3352

Closed
mischah opened this issue Sep 27, 2016 · 3 comments
Closed

npm shrinkwrap produces error with hapi #3352

mischah opened this issue Sep 27, 2016 · 3 comments
Assignees
Labels
Milestone

Comments

@mischah
Copy link

@mischah mischah commented Sep 27, 2016

Hej there,

I’m trying to shrinkwrap a hapi project and I’m able to reproduce that issue with hapi as single dependency.

Steps to reproduce**

mkdir hapi-shrinkwrap
cd hapi-shrinkwrap
npm init -f
npm i hapi --save
npm shrinkwrap

What was the result you received?

The following npm error:

npm ERR! invalid: have wreck@9.0.0 (expected: 8.x.x) /Users/mkuehnel/hapi-shrinkwrap/node_modules/hapi/node_modules/subtext/node_modules/wreck
npm ERR! extraneous: wreck@9.0.0 /Users/mkuehnel/hapi-shrinkwrap/node_modules/hapi/node_modules/subtext/node_modules/wreck

What did you expect?

That npm creates a npm-shrinkwrap.json file as described in the npm docs.

Context

  • node version: 5.5.0 as well as 6.5.0
  • hapi version: 15.1.0
  • os: OS X 10.11.6
  • npm version: 3.10.3
@mischah
Copy link
Author

@mischah mischah commented Sep 27, 2016

Uhm. Using the following succesfully creates a npm-shrinkwrap.json

npm prune && npm update && npm install && npm shrinkwrap

Based on http://stackoverflow.com/questions/25696584/npm-list-giving-npm-err-invalid-output

@Marsup
Copy link
Contributor

@Marsup Marsup commented Sep 27, 2016

It's already fixed by #3351, just need to merge it.

@mischah
Copy link
Author

@mischah mischah commented Sep 27, 2016

Thanks. Feel free to close the issue whenever you want 👌

@Marsup Marsup added the bug label Sep 27, 2016
@hueniverse hueniverse self-assigned this Sep 27, 2016
@hueniverse hueniverse added this to the 15.1.1 milestone Sep 27, 2016
@lock lock bot locked as resolved and limited conversation to collaborators Jan 10, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

3 participants