Problem/Motivation

In #3252088: Increase Drupal::MINIMUM_PHP to 8.0.0 we bumped Drupal 10.0.x to PHP 8.0.0.

We have since discovered some Symfony components require PHP 8.0.2 or above:

$ composer why-not symfony/event-dispatcher-contracts 3
symfony/event-dispatcher-contracts  v3.0.0  requires  php (>=8.0.2 but 8.0.0 is installed) 

Steps to reproduce

Proposed resolution

Bump MINIMUM_PHP and related constants (e.g. in composer.json) to 8.0.2.

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Drupal 10's minimum PHP version requirement is PHP 8.0.2. 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
#14 3255271-14.patch4.78 KBlongwave
#5 3255271-5.patch4.78 KBspokje
#4 3255271-4.patch4.48 KBspokje

Comments

longwave created an issue. See original summary.

longwave’s picture

Title: 2Increase Drupal::MINIMUM_PHP to 8.0.2 » Increase Drupal::MINIMUM_PHP to 8.0.2
longwave’s picture

Symfony chose 8.0.2 because that was the version used on TravisCI, so that doesn't really affect us, but in order to be compatible we need the same or higher minimum version.

https://github.com/symfony/symfony/pull/41282#issuecomment-845030355

spokje’s picture

StatusFileSize
new4.48 KB
spokje’s picture

StatusFileSize
new4.78 KB
spokje’s picture

Issue summary: View changes
Issue tags: +10.0.0 release notes
spokje’s picture

Status: Active » Needs review
longwave’s picture

Status: Needs review » Reviewed & tested by the community

Patch changes everything that was changed in the 8.0.0 upgrade, this looks correct to me. Thanks!

  • catch committed d87707c on 10.0.x
    Issue #3255271 by Spokje, longwave: Increase Drupal::MINIMUM_PHP to 8.0....
catch’s picture

Status: Reviewed & tested by the community » Fixed

Committed d87707c and pushed to 10.0.x. Thanks!

gábor hojtsy’s picture

Note that after this commit, Drupal 10 can only be tested on PHP 8.1 due to DrupalCI not having 8.0.2. Opened #3255439: Update PHP 8.0.0 to PHP 8.0.13 on DrupalCI to allow Drupal 10 testing on PHP 8.0.x.

  • catch committed 45e75ce on 10.0.x
    Revert "Issue #3255271 by Spokje, longwave: Increase Drupal::MINIMUM_PHP...
catch’s picture

Status: Fixed » Reviewed & tested by the community

Let's roll this back until the environment is there.

longwave’s picture

StatusFileSize
new4.78 KB

@mixologic reported in Slack that 8.0.13 should be used now, rerolled #5.

xjm’s picture

This will also need docs updates on the system requirements page:
https://www.drupal.org/docs/system-requirements/php-requirements

xjm’s picture

xjm’s picture

xjm’s picture

Issue summary: View changes

  • xjm committed 3bca95b on 10.0.x
    Issue #3255271 by Spokje, longwave, catch: Increase Drupal::MINIMUM_PHP...
xjm’s picture

Status: Reviewed & tested by the community » Fixed

And, committed #14. Thanks!

Adding credit for @Gábor Hojtsy for flagging the testbot issue.

xjm’s picture

Published the CR.

Status: Fixed » Closed (fixed)

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