search

gruntjs / grunt-contrib-sass

Compile Sass to CSS.
http://gruntjs.com/
MIT License
848 stars 141 forks source link

allow sass binary path to passed #220

Closed nick-less closed 8 years ago

sindresorhus commented 8 years ago

No thanks. Just add it to your $PATH. That way it will work for any other tool using it too.

nick-less commented 8 years ago

hm, we have setups where using the path is not possible (e.g frontend builds using maven-frontend-plugin, where build dependencies (node, grunt, sass..) are installed on the fly). No chance to add these three lines?

sindresorhus commented 8 years ago

There's always a way. You could also just modify process.env.PATH in the gruntfile.

It's never "just adding 3 lines". It requires docs, tests, and creates overhead for the user that has to understand it.

nick-less commented 8 years ago

Ok modifing process.env.PATH would allow me to add the path, but it wont allow me to invoke "node-sass" instead of "sass"...

sindresorhus commented 8 years ago

node-sass is not compatible anyways. Use https://github.com/sindresorhus/grunt-sass if you want node-sass.