-
Notifications
You must be signed in to change notification settings - Fork 39
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
PHP 7 support #30
Comments
yes. I will start porting in the next month or two unless somebody wants to send a PR :-) |
PHP7 shouldn't be that hard to refactor for. I'm midway through already. |
Great, thanks for the info! |
Well I have a successful build for PHP7: https://travis-ci.org/absalomedia/sassphp/jobs/90245122 I'll start cherry picking stuff ready for a PR |
Great news! We'll test it once it's ready. best regards Mark On 10 November 2015 at 05:35, Lawrence Meckan [email protected]
|
Hello. Any idea about when this will be merged? It's the only thing stopping me from putting PHP7 into production. If you need me to help test it or anything please let me know. Regards, Dane |
Only thing I've had repeated issues on is Travis segfaulting when trying to run PHPT data. It builds ok, passes all tests locally it's just Travis don't like my test setup for some reason. |
That's great news! Unfortunately git/github is not my normal stomping ground so I need some guidance. Where can I get your sources so I can build it and test my program against it? |
Here: https://github.com/absalomedia/sassphp I'm confident it builds successfully under PHP7. |
I cloned https://github.com/absalomedia/sassphp repository and the build was successfully. But when i run this simple script I receive a segmentation fault:
This is test.scss
I'm building against php 7.0.1. Alex |
This is from the PHPT file that tests file compilation:
You shouldn't need to a file_get_contents call to get the output. |
Sorry, you are right, file_get_contents is not necessary. But the segmentation fault is happen anyway. |
Hello, Any progress on the segmentation fault? I'm trying to go live w/ PHP7 next week Tuesday and I don't have a work around for this so any progress would be greatly, greatly, greatly appreciated! |
I'm building on Debian & Travis. I'm still working on getting stack traces from them. It might help if I had the stack dump from @asterixcapri as well. |
Just provide some instructions on exactly what it is you need me to do Thanks, Dane On Wed, Jan 13, 2016 at 7:02 PM, Lawrence Meckan [email protected]
|
|
OK. I've done a minor rewrite based on the feedback from @asterixcapri & it does build without errors on Travis. I still can't get any segfault data out of Travis when tests are being run though. Pull the latest & see what it does. |
I tried the new version and it still segfaulted. So I've gone one step more than @asterixcapri in the hope that it will help you root cause and fix this. I compiled a debug build of PHP7.0.2 so that you'll have more detail. Please note, I've used the same test script that @asterixcapri used. I've attached the output as a text file because I haven't figured out how to stop github from screwing up the formatting. I've also attached the core file in case you want to root around in it yourself. Good luck! |
@studdugie thanks for that. The more backtraces I have, the better. Looks like the changes to the Zend Hashtable API are breaking it in PHP7. |
With the new version now it outputs the correct result but then segfaults anyway... |
@asterixcapri that's understandable with the changes to the Hashtable API. I think I've fixed up the Hashtable / object issue now. Travis isn't segfaulting as badly now in my latest runs |
Now have an updated pull request with Libsass 3.3.3 & latest raft of PHP7 changes #32 |
Just built this and ran
|
It's still segfaulting. |
Hi, I put this in the Pull Request, but I figured I would add it to this thread as well. Is there any chance we can get a version of the current library that uses libsass 3.3.3 without the php7 compatibility? We are trying to use the latest version of Bourbon and Neat, but they require some things in the latest version of libsass. Michael |
@mandersondesign the way the codebase is designed is that it negotiates between PHP below version 7 & version 7 and above for the same functionality. Pulling https://github.com/absalomedia/sassphp should get you a stable SASS extension that builds under PHP 5.4 to 5.6 with Libsass 3.3.3. |
OK. We now have a stable PR with PHP7 if anyone wants to merge it in? |
+1 I was able to test the pull request from @absalomedia and had no problems. Would love to see this pull request go through. |
Same here, I can confirm the PR is working just fine with PHP7 |
+1 |
1 similar comment
+1 |
Hi,
Do you have any plans to work on the extension for PHP 7?
regards
Mark
The text was updated successfully, but these errors were encountered: