Optional dependencies not met

[aderaaij]

Do you want to request a feature or report a bug?
A bug

What is the current behavior?
When running yarn optional dependencies of packages are not met.

If the current behavior is a bug, please provide the steps to reproduce.
Run a package with optionalDependencies declared
What is the expected behavior?
Since there is an explicit --ignore-optional flag, I expect optional dependencies to be met.

Please mention your node.js, yarn and operating system version.
node: 4.1.2
yarn: 0.15.1
osx: 10.12

[Jessidhia]

--ignore-optional is for not even attempting to install them. By default, they should be attempted to be installed, but not complain on failure.

[sebmck]

Do you have any output that would imply that any of these failed to install or is it silent? If you put together a repro that I can run locally that reproduces this then I should definently be able to fix this!

[aderaaij]

Woah, that's a fast response!

The package at hand is gulp-imagemin and it seems to have optional dependencies that are not truly optional. The packages are not installed in the node_modules folder and running the task gives an error. Could you give it a try with gulp-imagemin or should I make a quick example?

[rdmurphy]

I'm hitting the same issue via gulp-imagemin. It throws this error when I try to run my gulp project that uses that package:

Error: Cannot find module 'imagemin-gifsicle'
    at Function.Module._resolveFilename (module.js:455:15)
    at Function.Module._load (module.js:403:25)
    at Module.require (module.js:483:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/Users/path/to/project/node_modules/gulp-imagemin/index.js:8:26)
    at Module._compile (module.js:556:32)
    at Object.Module._extensions..js (module.js:565:10)
    at Module.load (module.js:473:32)
    at tryModuleLoad (module.js:432:12)
    at Function.Module._load (module.js:424:3)
    at Module.require (module.js:483:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/Users/path/to/project/gulp/images.js:5:18)
    at Module._compile (module.js:556:32)
    at Object.Module._extensions..js (module.js:565:10)
    at Module.load (module.js:473:32)

I'm not sure if this is a gulp-imagemin issue or imagemin itself.

[thasmo]

Same here. Looks like optional dependencies won't get installed.

node 6.7.0
yarn 0.15.1
windows 10 64-bit

[deepu105]

same here for gulp-imagemin

module.js:341                                                                                                 
    throw err;                                                                                                
    ^                                                                                                         

Error: Cannot find module 'imagemin-gifsicle'                                                                 
    at Function.Module._resolveFilename (module.js:339:15)                                                    
    at Function.Module._load (module.js:290:25)                                                               
    at Module.require (module.js:367:17)                                                                      
    at require (internal/module.js:16:19)                                                                     
    at Object.<anonymous> (D:\Projects\test\angular2test\node_modules\gulp-imagemin\index.js:8:26)            
    at Module._compile (module.js:413:34)                                                                     
    at Object.Module._extensions..js (module.js:422:10)                                                       
    at Module.load (module.js:357:32)                                                                         
    at Function.Module._load (module.js:314:12)                                                               
    at Module.require (module.js:367:17)                                                                      

[jing2si]

@deepu105 04d2a3d

[sendilkumarn]

@deepu105 if the PR is accepted. It will close the issue

@jing2si 👍