-
Notifications
You must be signed in to change notification settings - Fork 98
Error when compiling sass/scss #14
Comments
Ugh, this is a node-sass bug, you can try using https://github.com/electronjs/electron-rebuild - node-pre-gyp sucks with Electron right now |
Unfortunately electron-rebuild did not help. |
After further investigation, it the origin of these issues is actually node-sass. |
@eduardhojbota If there is a node-sass bug open, can you link it here? |
There isn't one at the moment, so I will create one and post it here. P.S. sass/node-sass#1047 |
@eduardhojbota Thanks! |
Looks like we've got an answer pretty quickly. You might want to check it out. |
Considering that node-sass does not offer support for electron (atom-shell) I shall reopen this issue. |
I have a plan to fix this, 🔜 |
Any word on a resolution for this issue? |
@jgeurts Running Electron v0.33.1 and electron-compile v0.9.2, this seems to work correctly for me now at least. |
@AnthonyPorthouse, could you please specify your operating system as well? |
I am getting the same error, but with ZeroMQ (zmq). Running |
@eduardhojbota Linux x86-64, but I was having the same issue previously. I'll try it on windows once I get back from work. Edit: Ok yeah, still having this problem on Windows 10 |
Sass is removed from the newest version of electron-compile, sorry :( |
But if the node-sass was working for Unix, why remove it entirely? |
@eduardhojbota In the latest version of node-sass, it breaks Linux too for everyone, even if you're not using node-sass, by including a module that uses |
So I stumbled upon this issue when trying to get node-sass to work in electron myself. I figured out a way to do it on my system (Debian Jessie). Using nvm I switched to the version of node that electron is using, currently 5.1.1, and installed node-sass. It worked flawlessly after that. Just figured that I'd share my solution. |
@Aldlevine That's a great first step. For me it works as well on Xubuntu. When create my app on Xubuntu and run it on Xubuntu it works but it does not work if I want to distribute the app for Windows or OSX. Do you know how to solve this for other platforms as well? What are your thoughts on that? |
@paulcbetts so, if I understand right; the solution merged in with this PR |
@Aldlevine I've tried the same for windows but that did not solve the problem. :( |
Now when compiling a valid scss/sass file, the following error is received:
Platform: Windows 8.1 x64
P.S. It works on OSX 10.10 tho
The text was updated successfully, but these errors were encountered: