Skip to content

Upgrade to 3.4.9 #232

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

Merged
merged 1 commit into from
Feb 6, 2018
Merged

Upgrade to 3.4.9 #232

merged 1 commit into from
Feb 6, 2018

Conversation

asottile
Copy link
Member

@asottile asottile commented Feb 5, 2018

No description provided.

@asottile
Copy link
Member Author

asottile commented Feb 5, 2018

yay, sourcemaps changed -- this makes sense now that they're unicode aware (reading through changelogs) -- I'll go update them

@asottile
Copy link
Member Author

asottile commented Feb 5, 2018

hrmmm though segfault on posix -- seems like there might be an upstream bug here... trying to get a trace...

@asottile
Copy link
Member Author

asottile commented Feb 5, 2018

trying to get a core dump but not doing so well -- I'm not on my usual computer.

@xzyfer just in case it is actually a problem -- we're seeing segfaults with 3.4.9 that we weren't seeing with 3.4.8 -- I'll try and make a minimal reproduction

@xzyfer
Copy link
Contributor

xzyfer commented Feb 6, 2018 via email

@mgreter
Copy link
Contributor

mgreter commented Feb 6, 2018

I had to delete the first tagging since I saw the same error on perl-libsass (it was maybe up for 30 minutes). Are you sure you have sass/libsass@6de5050 included?

Sorry for the inconvenience.

@asottile
Copy link
Member Author

asottile commented Feb 6, 2018

ah nice -- I was too quick! I'll try again with the new tag -- thanks 🎉

@mgreter
Copy link
Contributor

mgreter commented Feb 6, 2018

Cool, I guess we should add some tests in libsass directly to catch these errors.
Unfortunately SassC doesn't use the more advanced API features ...

@asottile
Copy link
Member Author

asottile commented Feb 6, 2018

awesome -- let me know if you'd like some help with that :)

@mgreter
Copy link
Contributor

mgreter commented Feb 6, 2018

IMO we would need something like https://github.com/sass/libsass/blob/master/docs/api-function-example.md for travis and appveyor to test custom functions/importers directly. I don't see any other easy way to cover this in our CI otherwise. If you can come up with a PR that would be great ;)

@asottile
Copy link
Member Author

asottile commented Feb 6, 2018

I'll see what I can cook up in my copious amounts of freetime, I've added it to the top of my "todo" list :)

@asottile asottile merged commit dbb4ea4 into master Feb 6, 2018
@asottile asottile deleted the 3_4_9 branch February 6, 2018 01:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants