-
Notifications
You must be signed in to change notification settings - Fork 75
Port PR #91 to 2.7 #94
Comments
Are there any reasons for your request? |
Yes: the concrete5 CMS uses version 2.7.7, but it doesn't work on PHP 7.3 because of the problem fixed in #91. |
@mlocati
PHP 5.5 is dead since 21 Jul 2016 – 2 years, 2 months ago!
I'm sorry, but if we would support you, then the next wrong signal for this user base would be given: stay on unsupported versions and do nothing.
Version 3 of zend-stdlib still supports 5.6, but with the end of the year version 5.6 and 7.0 are also dead. |
@froschdesign Unfortunately since we are bound by our release cycle and semver we cannot drop support for PHP 5.5.9 in a minor version update and so we cannot upgrade away from Sounds like the only way forward is for concrete5 to fork and deprecate so that we can maintain support of the PHP language for our actively supported version. |
That simply means that your LTS release has an upper non-inclusive bound of PHP 7.3.0 Much simpler solution, much less pain in general.
|
This does not solve the problem that you supporting dead PHP versions. It makes the situation worse! |
If we raise the minimum PHP required version in a dot release, I'm pretty sure I'll have to explain that a few thousands of times (literally). |
@mlocati |
That's fair, but just not the path we want to take unless we have no other choice. I think it's completely reasonable that you do not want to make this change, we just want to make sure that our in-progress minor release supports as much as possible. With that in mind, we've already taken the steps to fork this repo so that's the path we're likely to take unless something changes here.
This is something that we have not considered, but might be worth discussing further in our own circle. I would absolutely argue that raising a minimum version requirement is a breaking change and so should not ever be included in a minor version release, but clearly there are people that disagree with me there. I appreciate the consideration from you all on this issue, I think we've pretty well solved it at this point. |
is it possible to create a new release for the 2.7 series with #91 included?
The text was updated successfully, but these errors were encountered: