-
Notifications
You must be signed in to change notification settings - Fork 7
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 8.0 compatibility #21
Comments
same error with 8.1. |
I just put in a pull request to get the extension working past 8.0. I’ve tested it on 7.4, 8.0 and 8.1, and they all seem to work. |
after Installing extension=cmark.so PHP doesn't start: PHP Fatal error: Class CommonMark\Node must implement interface Traversable as part of either Iterator or IteratorAggregate in Unknown on line 0 PHP 8.1.13 (cli) (built: Nov 26 2022 14:07:36) (NTS) |
@krakjoe @remicollet @weltling Sorry to highlight all of you, but this extension is effectively dead. The currently released version is not compatible with any supported PHP version at this time. There are open pull requests to fix compatibility. If one of you could take the time to review them, we can get this extension alive again. Otherwise it's dead without someone forking it. |
The pull request that I submitted passed all tests with 8.0 and 8.1, but some tests failed with 8.2 and the 8.3 release candidate. I was able to fix most of them with some minor adjustments to the expected output, but one test (./Node/Abstract/027.phpt) is still failing, and fixing it is beyond what I’m capable of. If anyone could look at it, I would appreciate it deeply. |
@anthonyryan1 @dwo0 You can download the DDL with patch for PHP 8 here: https://phpext.phptools.online/extension/text/cmark-360 |
It compiles against PHP 8.0 but it's failing to allow php-fpm to start.
Some of the other modules I interact with have used this release as a chance to switch to stub files (compiled with php-src/build/gen_stub.php) to automatically compile the function headers.
Should we follow suit on that? Or keep things as is as much as possible and only implement Traversable support.
The text was updated successfully, but these errors were encountered: