-
Notifications
You must be signed in to change notification settings - Fork 156
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Crash on syntax errors (2.1.0pre2 and later) #136
Comments
could you share the crash report? it doesn't crash on my environment.
|
Seems like only when I run in my Rails app (console or server) I get the crash. Running from irb gives the proper result. There is no information provided. The process just sends me back to the shell prompt.
|
Stepping through the code, I can get as far as the call to Native.compile_data_context(data_context) before it crashes. |
It only happens when mysql2 (I'm using 0.5.2) is loaded. Without mysql2:
With mysql2:
|
|
@glebm any thoughts on this? |
I am able to reproduce this. Interestingly, this only happens if Moving |
More detailed backtrace (obtained by removing
Looks like it never gets to the |
Works just fine when compiling from source. Only an issue with the pre-compiled gem. |
Our pre-compilation container uses GCC 5. Perhaps there is some incompatible change between the two versions. |
|
Inline exception constructor/destructor implementation seems to be a C++ pitfall, leading to ODR violations and being unable to catch exceptions in certain situations. This is a fix for a bug originally reported in: sass/sassc-ruby#136
Inline exception constructor/destructor implementation seems to be a C++ pitfall, leading to ODR violations and being unable to catch exceptions in certain situations (https://stackoverflow.com/a/3481427). This is a fix for a bug originally reported in: sass/sassc-ruby#136
Disabling While upgrading Disabling I'll send a PR disabling LTO for precompiled gems. |
Disabling `lto` increases the precompiled gem size (1.6M -> 8.6M). Fixes sass#136
2.2.0 release with no precompiled linux gem. |
Ruby 2.3.8
sassc 2.1.0.pre3 (and 2.1.0pre2)
Ubuntu 18.04.2 LTS
I get a crash when feeding sassc invalid CSS. No crash when given good CSS.
No crash on 2.0.1 or 2.1.0pre1.
The text was updated successfully, but these errors were encountered: