ember-cli-sass 3.3.0 is broken

[Trendy]

ember-cli-sass 3.3.0 tries to use an older version of node-sass that doesn't build properly on Linux.

Upgrading to ember-cli-sass 5.x.x seems to fix the issue for me, and still works as expected.

Should we upgrade the ember-cli-sass version to 5.4.0?

Errors on install:

> [email protected] install /home/jeremiah/xxxxx/node_modules/node-sass
> node scripts/install.js

Can not download file from https://raw.githubusercontent.com/sass/node-sass-binaries/v2.1.1/linux-x64-node-5.7/binding.node

> [email protected] postinstall /home/jeremiah/xxxxx/node_modules/node-sass
> node scripts/build.js

module.js:341
    throw err;
    ^

Error: Cannot find module '/home/jeremiah/xxxxx/node_modules/node-sass/node_modules/pangyp/bin/node-gyp'
    at Function.Module._resolveFilename (module.js:339:15)
    at Function.Module._load (module.js:290:25)
    at Function.Module.runMain (module.js:447:10)
    at startup (node.js:141:18)
    at node.js:933:3
Build failed

That results in the following when I run ember server:

`libsass` bindings not found. Try reinstalling `node-sass`?
Error: `libsass` bindings not found. Try reinstalling `node-sass`?
    at getBinding (/home/jeremiah/xxxx/node_modules/broccoli-sass-source-maps/node_modules/node-sass/lib/index.js:22:11)
    at Object.<anonymous> (/home/jeremiah/xxxx/node_modules/broccoli-sass-source-maps/node_modules/node-sass/lib/index.js:188:23)
    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)
    at require (internal/module.js:16:19)
    at Object.<anonymous> (/home/jeremiah/xxxx/node_modules/broccoli-sass-source-maps/index.js:5:12)
    at Module._compile (module.js:413:34)

Work around

You can get around this by manually specifying which version of ember-cli-sass should be installed in your package.json file:

npm install --save-dev ember-cli-sass@latest

[robbyoconnor]

@Trendy except others won't do that -- it should be fixed if this project is properly maintained...

[Trendy]

@robbyoconnor I completely agree.

#286 should fix this issue and should probably be merged @mike-north.

[rynam0]

@mike-north: I know you're busy with other priorities but this one hung me up several times and it's definitely hanging others up. Could you please merge this one when you get a moment?

[pinx]

+1