workaround for https://github.com/sass/node-sass/issues/337

[tony-kerz]

i was getting the following until i followed some hints in this thread: sass/node-sass#337

warlock-scratch-2 $ warlock

Starting tasks: default
Starting task: flow::templates-to-build
Starting task: flow::coffee-to-build
Starting task: flow::sass-to-build
Starting task: flow::assets-to-build
Starting task: clean::styles-to-build
Starting task: flow::vendor-styles-to-build
[vendor-styles-to-build] Got empty globbing pattern. Skipping flow.
Starting task: flow::styles-to-build
Starting task: clean::scripts-to-build
Starting task: flow::vendor-scripts-to-build
Starting task: flow::scripts-to-build
Starting task: clean::html-to-build
Starting task: clean::scripts-to-compile
Starting task: flow::assets-to-compile
Starting task: clean::styles-to-compile
Starting task: clean::html-to-compile
Assertion failed: (val->IsString()), function _NanGetExternalParts, file ../node_modules/nan/nan.h, line 1725.
Abort trap: 6

[joshdmiller]

With ngbp, a lot of people requested source maps. I quickly perused the issue you mentioned, and there was a lot going on. To help me understand a little bit better, is this an upstream bug?

[tony-kerz]

yeah, source maps are def a goal, and yeah, this looks like what i'm sure will be a transient issue with an upstream dependency. so depending on upstream fix versions, it may just resolve itself with the chain: gulp-sass (~0.7) -> node-sass (^0.9) -> nan (~1.2.0).

as i think you are alluding to, it may not make sense to incorporate this pull to accommodate a transient situation upstream, but i figured i would just post it in case someone was plagued by the issue and wanted to use it for a temporary workaround while upstream issues got sorted out. feel free to handle as you see fit while this situation exists.

regards,
tony.

[joshdmiller]

as i think you are alluding to, it may not make sense to incorporate this pull to accommodate a transient situation upstream

You're right, with the exception that this is a default configuration option that is not required to function. I'm open to making the change, I just want to understand what's going on a little better.

Under what circumstances will this error occur?

[tony-kerz]

nothing too fancy as i'm kind of a novice, pretty sure i just:

• added the sass spell like so: https://github.com/tony-kerz/warlock-test/blob/master/package.json#L19
• added a couple of simple sass files like so: https://github.com/tony-kerz/warlock-test/tree/master/src/app

and ran warlock

[joshdmiller]

Alright, sold. :-)

Landed as 3c33977.