search

joliss / broccoli-sass

Sass compiler for Broccoli, using libsass
MIT License
44 stars 97 forks source link

Ember-cli broccoli-sass heisenbug #42

Closed michaelBenin closed 9 years ago

michaelBenin commented 9 years ago

This happens sporadically when running ember test.

Version: 0.3.3

ember --version version: 0.1.2 node: 0.10.33 npm: 2.1.3

Error Message: Building... Module did not self-register. Error: Module did not self-register. at Error (native) at Module.load (module.js:355:32) at Function.Module._load (module.js:310:12) at Module.require (module.js:365:17) at require (module.js:384:17) at Object. (/Users/michaelbenin/Projects/product-timeline/node_modules/broccoli-sass/node_modules/node-sass/lib/index.js:181:15) at Module._compile (module.js:460:26) at Object.Module._extensions..js (module.js:478:10) at Module.load (module.js:355:32) at Function.Module._load (module.js:310:12) at Module.require (module.js:365:17) at require (module.js:384:17)

simonexmachina commented 9 years ago

broccoli-sass is a thin wrapper around node-sass, and this error's coming from native code which means that it's definitely a node-sass issue. I suggest you create an issue there. Feel free to reopen if you think not.

michaelBenin commented 9 years ago

I don't understand how closing this issue helps the project. If broccoli-sass relies on a broken module this issue should remain open until that module is fixed.

simonexmachina commented 9 years ago

I disagree, I think the issue should exist on the project that can do something to get it resolved - there's nothing that can be done by broccoli-sass to resolve this issue, so the issue should be closed. If the fix in node-sass requires an update to this project then that's a separate issue, but in all likelihood it will be a minor or patch version which this module will get without requiring any changes, thanks to the wonders of semver.