Bug #80816 [Com]: Missing symbol spl_ce_Countable
| From: | kocsismate@php.net | Date: | Mon, 01 Mar 2021 17:48:55 +0000 |
| Subject: | Bug #80816 [Com]: Missing symbol spl_ce_Countable | ||
| References: | 1 | Groups: | php.bugs |
| Request: | Send a blank email to php-bugs+get-232457@lists.php.net to get a copy of this message | ||
Edit report at https://bugs.php.net/bug.php?id=80816&edit=1
ID: 80816
Comment by: kocsismate@php.net
Reported by: jeviwi3279 at timothyjsilverman dot com
Summary: Missing symbol spl_ce_Countable
Status: Assigned
Type: Bug
Package: Compile Failure
Operating System: macOS
PHP Version: master-Git-2021-03-01 (snap)
Assigned To: kocsismate
Block user comment: N
Private report: N
New Comment:
To be honest, I'd prefer simply adding the related changelog entry, and not to restore this
alias (and the few other ones). These aliases were only needed due to technical reasons, since SPL
did some macro magic, like this: https://github.com/php/php-src/pull/6709/files#diff-f86ee648d398a68130b2bff8808bcdb6382278eb198f98c5f6e12cb1f051a23eL1878
Such breaking changes (which affect extension internals) can happen in minor releases. Of course, we
don't want to cause very large BC breaks, but this one seems like a very small one for me. Are
you sure that a lot of extensions are affected?
Previous Comments:
------------------------------------------------------------------------
[2021-03-01 13:34:33] jeviwi3279 at timothyjsilverman dot com
Yes, it will be a breaking change for many extensions when they are compiled for PHP 8.1.
If it is not much effort, it should be restored now and removed later in a major release. Otherwise
please add it to the changelog.
------------------------------------------------------------------------
[2021-03-01 13:21:57] nikic@php.net
It did get dropped, but it might make sense to restore it?
------------------------------------------------------------------------
[2021-03-01 13:02:11] jeviwi3279 at timothyjsilverman dot com
Description:
------------
Has the alias spl_ce_Countable for zend_ce_countable in SPL removed in master. I could not find this
change in the changelog. This would be a breaking change for many extensions.
------------------------------------------------------------------------
--
Edit this bug report at https://bugs.php.net/bug.php?id=80816&edit=1