Problem/Motivation

As we are targetting Symfony 6 in Drupal 10.0.0, we need to increase our minimum PHP version to 8.0.

Before release this may be increased further to 8.1 as per #3173787: [policy] Require PHP 8.1 for Drupal 10.0 if a dependency does, otherwise require PHP 8.0. (up from 7.3 in Drupal 9).

Steps to reproduce

Proposed resolution

Remaining tasks

Do it.
Remove any code that refers to PHP 7.

User interface changes

API changes

Data model changes

Release notes snippet

Drupal 10's minimum PHP version requirements was raised to PHP 8.0.0. It will not be possible to run Drupal 10 on PHP 7 or earlier. However, there is a possibility that before Drupal 10's release, the PHP requirement will be raised again to PHP 8.1. See #3173787: [policy] Require PHP 8.1 for Drupal 10.0 if a dependency does, otherwise require PHP 8.0. (up from 7.3 in Drupal 9) for ongoing discussion.

CommentFileSizeAuthor
#6 3252088-6.patch5.01 KBGábor Hojtsy
#3 3252088-3.patch5.04 KBlongwave
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

longwave created an issue. See original summary.

longwave’s picture

longwave
he/him
Primary language English
Location UK
CreditAttribution: longwave commented
Parent issue: » #3118147: [meta] Set Drupal 10 platform and browser requirements six months before the release

As per the parent meta, "Require at least PHP 8.0 as soon as 10.0.x is opened."

longwave’s picture

longwave
he/him
Primary language English
Location UK
CreditAttribution: longwave commented
Status: Active » Needs review
FileSize
3252088-3.patch5.04 KB
andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost as a volunteer and at Skilld commented 
+++ b/core/INSTALL.txt
-- PHP 7.3.0 (or greater) (http://php.net/). For better security support it is
-  recommended to update to at least 7.3.13.
+- PHP 8.0.0 (or greater) (http://php.net/).

maybe there's some recommented version like 8.0.6 https://www.php.net/ChangeLog-8.php#8.0.6

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch at Third and Grove commented
Status: Needs review » Reviewed & tested by the community

I think 8.0.0 is fine - there's a good chance we'll raise this again anyway.

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy at Acquia commented
FileSize
3252088-6.patch5.01 KB

Patch did not apply. First posting a reroll for testing.

  • 401a725 committed on 10.0.x 
    Issue #3252088 by longwave, Gábor Hojtsy, catch: Increase Drupal::...
Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy at Acquia commented
Status: Reviewed & tested by the community » Fixed

Green, got it in :) Thanks @longwave!

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy at Acquia commented

I also went to remove any testing environmens that were using less than PHP 8, but there were no such environments (of course) for Drupal 10. So nothing to update there.

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy at Acquia commented
Issue summary: View changes

Also added a change record at https://www.drupal.org/node/3253062

Copied that to the release notes snippet as well.

xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm at Acquia commented
Parent issue: #3118147: [meta] Set Drupal 10 platform and browser requirements six months before the release » #3251854: [META] Requirements for tagging Drupal 10.0.0-alpha1
longwave’s picture

longwave
he/him
Primary language English
Location UK
CreditAttribution: longwave commented
Issue tags: +10.0.0 release notes

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.