Skip to content
This repository has been archived by the owner on Aug 11, 2022. It is now read-only.

optionalDependencies is really wonky #2737

Closed
ghost opened this issue Aug 20, 2012 · 1 comment
Closed

optionalDependencies is really wonky #2737

ghost opened this issue Aug 20, 2012 · 1 comment

Comments

@ghost
Copy link

ghost commented Aug 20, 2012

I've been running into problems where an optionalDependencies entry is specified but npm treats it exactly like a regular dependency, which causes the whole install to fail instead of ignoring the optional package. This feature seems really weird. npm should just ignore optionalDependencies outright and if people want extra magic they can fiddle with the packages themselves.

@isaacs
Copy link
Contributor

isaacs commented Aug 21, 2012

The "try-and-continue-on-failure" behavior seems to be not surprising to most people. I think you had an old version that only continued on certain types of failures, but not others, which was the problem you ran into.

An opt-out would be nice.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant