jlitola / play-sass

Sass asset handling for Play Framework 2.0 *DEPRECATED*
MIT License
90 stars 28 forks source link

Won't work on windows #2

Closed kallebertell closed 11 years ago

kallebertell commented 11 years ago

Heya, thanks for the plugin.

I need to support development on windows and I used your plugin which at least worked great on a mac but it explodes on windows.

Seems to be an issue with ProcessBuilder not being able find "sass.bat" when the command given is "sass". I forked and tried replaced "sass" with "sass.bat" and it started working on windows.

I just hopped into scala/play/sbt so I don't dare making the fix myself (yet) but it'd be cool if it'd check OS for the correct command , or maybe just make the command itself configurable.

jlitola commented 11 years ago

Hi,

Thanks for implementing fix for this in #3. I have now merged it in, and released 0.1.2 with those changes.