-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Evaluate dropping php7.0 support #12021
Comments
Ubuntu 18.04 is php7.2 if I'm not mistaken |
Debian Stretch is 7.0: https://packages.debian.org/stretch/php |
Debian will ship 7.2 in Buster: https://tracker.debian.org/pkg/php7.2 |
RHEL has a software collections for php7.1: https://www.softwarecollections.org/en/scls/rhscl/rh-php71/ |
SLES15 seems to have 7.2: https://build.opensuse.org/package/show/Apache:Modules/php7 |
besides it getting eol, are there any 7.1 features that people are waiting to make us of? |
More and more of our dependencies are dropping 7.0 support. So not upgrading blocks us from updating those. |
fair enough |
Nullable and void return types e.g. |
It's probably a good idea to be able to run the current Nextcloud version on a current Debian setup. Debian buster comes out in spring 2019. So until end of 2019 (to give some time for migration), it would be great to have php7.0 support so NC can be used with debian stretch. The current support cycle for a Nextcloud version seems to be around 1 year. So for NC 15 it would be great to still have php 7.0 support. From the survey tool, do you get any feedback which php versions are used? |
15 will support php7, 7.1 and 7.2 |
Current stats:
|
If Debian abandons 7.0 in 2019 only (when exactly?), we should perhaps keep it supported into 2020. |
But as @rullzer said, Nextcloud 15 is definitely gonna support PHP 7.0. |
True… well, depends how much pressure we want to put on those admins. As, I guess Debian, will support this for a long time on that release. They have their full freeze planned for March, which is still far away from a release. This it depends on our future release plannings, but maybe it is reasonable to keep the support until Debian releases. |
IF we decide to still support php7.0 after it is EOL I do vote for adding a notification to the admin settings. Because if there are significant security issues with it php won't fix them anymore |
I'm totally fine with adding a warning to all releases right now to get awareness to the admins. |
Actually not a bad idea. As even users on 13 on php7.0 end of december will run defacto EOL software |
How do we proceed for 16? |
Just ran into an issue: Mariadb 10.2 (which is used on many Docker images and other installations) is not fully supported by Found at nextcloud/twofactor_admin#20 (comment). |
Current stats:
|
NC16 will be >=PHP7.1 |
Added to #12915 |
Thank you all for the good work and the assessment. However, just as a note on: @MorrisJobke : |
@MorrisJobke Are those stats publicly available somwhere? |
Nope. |
According to https://secure.php.net/supported-versions.php, php7.0 EOL's soon. We should evaluate when to increase the minimum supported version.
cc @blizzz @daita @danxuliu @georgehrke @icewind1991 @juliushaertl @MorrisJobke @nickvergessen @rullzer @schiessle @skjnldsv
The text was updated successfully, but these errors were encountered: