concordusapps / grunt-haml

Process HAML templates to precompiled JavaScript or rendered HTML.
https://npmjs.org/package/grunt-haml
MIT License
41 stars 28 forks source link

Make execSync optional or use something else (or fix execSync) #16

Closed mehcode closed 11 years ago

mehcode commented 11 years ago

Here is the problem. I'm on a modern OS. python is python 3. execSync depends on node-gyp which requires python to be in the PATH and be python 2. node-gyp should really use python2 instead. Or we should use something else besides execSync. Something.. it's getting a bit annoying to install grunt-haml on my box.

joneshf commented 11 years ago

What is it even used for?

joneshf commented 11 years ago

Oh, I see, https://github.com/concordusapps/grunt-haml/blob/master/tasks/haml.js#L238 Can we use ruby-haml or something?

mehcode commented 11 years ago

Yeah that would work. We'd need to make the grunt task async (this.async() gives done callback) to implement it.