Skip to content
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

@extend causing a coredump #165

Closed
sveetch opened this issue Oct 14, 2016 · 3 comments
Closed

@extend causing a coredump #165

sveetch opened this issue Oct 14, 2016 · 3 comments

Comments

@sveetch
Copy link

sveetch commented Oct 14, 2016

Hi,

I found an obscure bug that cause a coredump, here you can see a sample source on SassMeister:

https://gist.github.com/sveetch/99fa86f9716b020d1cb19e869b20971a

Using it from commandline :

sassc sample.scss

It results in:

Segmentation fault (core dumped)

And nothing else in the output.

Basically i was trying to compile this using my Boussole project, so through your sass module and finally tried directly with sassc commandline tool to isolate the bug.

Finally i tried this sample code on SassMeister with any available compiler and always have a correct return output. Also i successfully compiled this code with Compass 1.0.1.

@asottile
Copy link
Member

This is likely fixed by 3.3.6 See sass/libsass#2017 sass/libsass#2031 which we cannot upgrade to due to sass/libsass#2046 (We tried to here: #149).

We'll probably have to wait until they drop 3.4 (hopefully soon!): sass/libsass#2174

I can confirm your bug:

$ sassc --version
sassc 0.11.1 (sass/libsass 3.3.5)
$ sassc test.scss 
Segmentation fault (core dumped)

I can also confirm that upgrading (when we can!) will fix it:

$ sassc --version
sassc 0.11.1 (sass/libsass 3.3.6-144-g50a6)
$ sassc test.scss
/*
 * Sample for proof of bug.
 *
 * It has been minimalized from original sources to find out the real stuff causing bug.
 */
.marked, .clean-block > .content h2.marked {
  text-decoration: underline; }

@media only screen and (min-width: lower-bound(64.063em, 90em)) {
  .clean-block > .content h2:not(.marked):not(.clean-block > .content h2.marked) {
    padding: 0; } }

@media only screen and (min-width: lower-bound(90.063em, 120em)) {
  .clean-block > .content h2:not(.marked):not(.clean-block > .content h2.marked) {
    font-size: 1rem; } }

Going to close this as it's an exact duplicate of an upstream bug and hopefully we'll be able to upgrade soon :)

@sveetch
Copy link
Author

sveetch commented Oct 14, 2016

Ah, i presumed something like this when readed another closed issues here. I'll keep watching about next update.

Thanks for your reply!

@asottile
Copy link
Member

3.3.6 is now available via 0.11.2 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants