Zopfli support

[webpack-contrib]

Zopfli support

[FredericHeem]

The PR creates 2 issues:

• Even if node-zopfli is not used, end users need to install python and a C++ compiler which is really annoying especially on windows.
• Behind a corporate firewall and with github blocked, node-zopfli cannot be installed.

What's your opinion of creating a new webpack plugin that supports node-zopfli and remove it from this package ?

[DNx5]

i have a problem with node-zopfli on windows too, after downloading and installing gigs of visual studio/sdk and other things but still can't get it to work for days. in the end I just manually change the version of node-zopfli inside compression-webpack-plugin's package.json into 2.0.0 and then it works fine.

[ShaneCourtrille]

Or the node-zopfli version dependency needs to be better maintained. The compiled binaries that would allow usage without python/VC++ for node v4 are only available for node-zopfli 2.x as per here

[ai]

@sokra https://www.npmjs.org/package/node-zopfli

[sokra]

PR? "optionalDependency" as it's a native module.

[ai]

Yeap, I will try to make PR but only after 23 of Oct (sorry :) ).

Maybe we should use it by default? In one hand it is slow. But it is good practice.

[stephanos]

@ai still interested?

[ai]

Nope, I moved to Gulp.

[generalov]

@DNx5 try to install VC++ and Python on Windows with https://www.npmjs.com/package/windows-build-tools (it takes about 5 minutes on my machine).

[FredericHeem]

Or add optional=false in your .npmrc to avoid installing unnecessary dependencies.

[michael-ciniawsky]

@FredericHeem @DNx5 @generalov @ShaneCourtrille If one of you guys still have issues regarding this plugin and windows useage please open a new issue 😛