Semantic-ui: [Installation] Gulp install fails on missing 'wrench' dependency for SUI 2.2.8

[andreyvk]

Hi,

First of all huge thanks for the 2.2.8 rollout! Amazing job!

I've found that there's a wrench dependency missing when doing gulp install.

Easy workaround would be to do a npm install --save wrench right after npm install fails at gulp install point.

Installing wrench globally doesnt work for some reason.

I am building SUI on Win 10. npm version 3.10.10, node version v6.9.5

[guzuomuse]

+1
Os:
win7
versions:
node v6.9.5
npm v4.1.2
gulp 3.9.1

[jbwl]

+1
OS: Mac OS Sierra

[jaimehing]

Is "adding wrench" as a dependency in the package the fix for this or wrench will be replaced since it's already deprecated?

+1 on your temporary fix @andreyvk. thanks!

[jpsirois]

I got the same issue with this env :

OS   : macOS Sierra 10.12.3 (16D32)
node : 7.5.0
npm  : 4.1.2
gulp : 3.9.1

[brianespinosa]

@jaimehing wrench should definitely be replaced since it is deprecated.

[jlukic]

I'll look into something for this tonight

[jlukic]

Looks like I forgot to change a path in the install script. Sorry, released fix as 2.2.9

[awgv]

The author of Wrench actually posted about it a few hours ago: https://github.com/Semantic-Org/Semantic-UI/issues/3404#issuecomment-281388078

[jlukic]

Wrench works well enough. I dont believe it's worth the testing to port the calls to a newer library at this time.