Problem/Motivation

Follow-up from #3258987: Status report should warn about OPcache bug in PHP 8.1.0 to 8.1.5.

We should increase RECOMMENDED_PHP on 10.0.x and perhaps 9.5.x to PHP 8.1.6 due to opcache/autoloader bugs in 8.1.0-8.1.5.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

PHP versions 8.1.0-8.1.5 have a bug with the PHP OPcache that may cause intermittent fatal errors at runtime for class autoloading. Therefore, the minimum recommended PHP version for Drupal 9.5 and above has been increased to 8.1.6.

Issue fork drupal-3294938

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

catch created an issue. See original summary.

xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm at Acquia commented
Version: 10.0.x-dev » 9.5.x-dev

I think we should backport this to 9.5, because it makes sense for 9.5 and 10.0 to have the same recommended PHP version.

xjm opened merge request !2490

xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm at Acquia commented
Issue summary: View changes

CR: https://www.drupal.org/node/3295061

xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm at Acquia commented
Status: Active » Needs review
Issue tags: +10.0.0 release notes, +9.5.0 release notes

Made the needed handbook update also: https://www.drupal.org/node/2891690/revisions/view/12666085/12706231

xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm at Acquia commented
Issue tags: +Drupal 10 beta should-have
dww’s picture

dww
we/he/they
CreditAttribution: dww at TEN7 commented
Status: Needs review » Reviewed & tested by the community

Looks great to me:

  1. Summary is clear and accurate.
  2. Trivial commit to make the stated change.
  3. I don't think we want/need any test changes for this.

RTBC

Thanks!
-Derek

  • catch committed c1a4dff on 10.1.x 
    Issue #3294938 by xjm, catch, dww: Increase RECOMMENDED_PHP to 8.1.6
  • catch committed fc74269 on 9.5.x
    Issue #3294938 by xjm, catch, dww: Increase RECOMMENDED_PHP to 8.1.6

(...

  • catch committed 465ffcf on 10.0.x 
    Issue #3294938 by xjm, catch, dww: Increase RECOMMENDED_PHP to 8.1.6

(...
catch’s picture

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

Committed/pushed to 10.1.x, 10.0.x and 9.5.x

For 9.4.x I've opened #3295085: [9.4 ONLY] Lower RECOMMENDED_PHP to 8.0 for the discussion. Marking this fixed but we can always re-open for backport if we decide we want to do that instead.

xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm at Acquia commented

Published the CR.

Status: Fixed » Closed (fixed)

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