Skip to content
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

composer.json PHP minimum Requirement => 7.x #691

Open
kkrieger85 opened this issue Jun 2, 2019 · 5 comments

Comments

4 participants
@kkrieger85
Copy link
Collaborator

commented Jun 2, 2019

Replaces #412

Update PHP minimum requirements to 7.1. for at least 1.9.4.x branch

Question: Also use php min req to 7.x for 1.9.3.x? We could add https://github.com/Inchoo/Inchoo_PHP7 to composer.json requirements

reference: https://php.net/supported-versions.php

@Flyingmana

This comment has been minimized.

Copy link
Member

commented Jun 5, 2019

there was a comment, that there are current LTS OperatingSystems shipping with a 5.x release, are they out of support now already?

@colinmollenhour

This comment has been minimized.

Copy link
Contributor

commented Jun 5, 2019

Ubuntu 14.04 LTS is now EOL (as of April 2019). Ubuntu 16.04 LTS uses PHP 7.0.

Do we care about RHEL/CentOS? I don't think we should because do we really want to support PHP 5.4 until June 2024?

@kkrieger85

This comment has been minimized.

Copy link
Collaborator Author

commented Jun 6, 2019

From my point of view, there is no need to support PHP 5.x.

There's the Inchoo/PHP7 package which brings PHP7 Support to Magento Core from 1.9.2.2. upon

With Magento Core 1.9.4.x this package was included in Magento Core

Besides that: We can strongly recommend using PHP7 at least for performance reasons

@colinmollenhour

This comment has been minimized.

Copy link
Contributor

commented Jun 6, 2019

My vote is for 7.0 to be the minimum since it isn't that old and is still supported by Ubuntu 16.04 LTS for two more years.

Debian 8 still uses 5.6 and will not be EOL for 11 more months, but there is an upgrade path to Debian 9 which uses 7.0.

Of course, for people using Docker (which I advocate should be everyone) which distro you run doesn't matter. The official docker images actually stopped supporting 7.0 already with 7.1 being the oldest getting updates.

@Flyingmana

This comment has been minimized.

Copy link
Member

commented Jun 7, 2019

then Iam fine with it, too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.