Remove dependency on node-sass about terraform HOT 4 CLOSED

Hey,

Thanks for opening an issue. Personally I wouldn’t characterise Node-sass or Libsass that way at all, in a relatively short period of time they’ve gone from a rough but workable implementation to something of a driving force in the Sass community. We bet on the Node-sass/Libsass combo fairly early and I think it’s proven to be the right choice. I’m not sure which Sass.js you’re referring to, but imposing Ruby as a dependency was less appealing to us than any of the other potential shortcomings.

Ideally, and in the case of SmartOS, the Libsass binaries should still get compile even on platforms where they’re not distrusted with Node-sass iteslf. If that’s not the case, we can help make some noise on specific issues, if you’d like? We’ve done that in the past and they took the concerns seriously.

If you’re having problems with Harp specifically, it’s possible that it’s because we’re not running the latest version, so that would be on us. We will definitely ship a release of Terraform once Node-sass v3.0.0 stable is out. Otherwise, you might be able to use npm shrinkwrap to change out or bump Node-sass manually.

Hope that’s helpful :)

from terraform.

I reviewed the past issues and didn't see other SmartOS problems. Sorry, maybe I was thinking of something else.

As it stands harp is 100% broken/unusable on SmartOS due to this issue. If you can make some noise for a fix in node-sass 2.x, that would be great.

from terraform.

Sorry if that was brash. It looks like I misunderstood how the fallbacks worked in Node-sass and the binary building is in the Node-sass v3.0.0 alpha. You should be able to opt-into it manually using npm shrinkwrap or it will be in a new release of Terraform once it’s out of alpha.

from terraform.

You weren't brash, I was the brash one – sorry about that.

Okay, I'll try again later. For now, checking in the compiled html and CSS
will make do.
On Apr 15, 2015 2:26 AM, "Kenneth Ormandy" [email protected] wrote:

Sorry if that was brash. It looks like I misunderstood how the fallbacks
worked in Node-sass and the binary building is in the Node-sass v3.0.0
alpha. You should be able to opt-into it manually using npm shrinkwrap or
it will be in a new release of Terraform once it’s out of alpha.

—
Reply to this email directly or view it on GitHub
#87 (comment).

from terraform.