Skip to content
This repository was archived by the owner on Jul 22, 2020. It is now read-only.

Comments

Drop fsevents from package-lock.json#163

Merged
prymitive merged 1 commit intomasterfrom
npm-install-linux
Aug 9, 2017
Merged

Drop fsevents from package-lock.json#163
prymitive merged 1 commit intomasterfrom
npm-install-linux

Conversation

@prymitive
Copy link
Contributor

This dependency was added to package-lock.json because npm install was initially run on MacOS and that package only works there. It spams CI jobs with warnings and keeps changing package-lock.json when developing on linux, MacOS builds should be fixed not to add it

@prymitive prymitive added the bug label Aug 8, 2017
@prymitive prymitive requested a review from terinjokes August 8, 2017 19:10
@terinjokes
Copy link

This looks like a pre-npm 5.3.0 bug. Failed installs of optional dependencies in npm >= 5.3.0 should be ignored.

@prymitive
Copy link
Contributor Author

https://travis-ci.org/#L296

npm install
npm WARN optional SKIPPING OPTIONAL DEPENDENCY: [email protected] (node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for [email protected]: wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})

It does ignore it, but it does it with some warnings and still keeps updating package-lock whenever I run npm install on linux (which is where majority of developments happen I think).

@prymitive
Copy link
Contributor Author

npm/npm#17722

This dependency was added to package-lock.json because npm install was initially run on MacOS and that package only works there. It spams CI jobs with warnings and keeps changing package-lock.json when developing on linux, MacOS builds should be fixed not to add it
Copy link

@terinjokes terinjokes left a comment

Choose a reason for hiding this comment

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

I'll allow it for now.

@prymitive prymitive merged commit 016f54e into master Aug 9, 2017
@prymitive prymitive deleted the npm-install-linux branch August 9, 2017 18:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants