Problem/Motivation

  • Composer 1 doesn't work on PHP 8, so DrupalCI uses Composer 2 to test on PHP 8.
  • DrupalCI uses Composer 2.0.3. It's possible that the problem described below is not an issue with Composer 2.0.7.
  • It appears that on Composer 2.0.3, the use of relative paths in ComposerProjectTemplatesTest::makeVendorPackage() causes an uncaught exception. See the failure for PHP 8 on #3156595-142: Make Drupal 9 installable on PHP8.

Proposed resolution

Write the absolute path rather than the relative path in the test fixture setup.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

effulgentsia created an issue. See original summary.

effulgentsia’s picture

effulgentsia’s picture

Crediting @alexpott, who figured this out in #3156595-146: Make Drupal 9 installable on PHP8.

effulgentsia’s picture

Status: Active » Reviewed & tested by the community

Since @alexpott wrote the patch, I'm eligible to RTBC it, so doing so.

alexpott’s picture

Version: 9.2.x-dev » 9.1.x-dev

This should be backported to 9.1.x - it is test only and helps us to ensure that that branch stays PHP 8 compatible regardless of what we decide for the 9.1 RC.

  • catch committed 45a21cc on 9.2.x
    Issue #3183825 by effulgentsia, alexpott: Use absolute instead of...

  • catch committed b63a325 on 9.1.x
    Issue #3183825 by effulgentsia, alexpott: Use absolute instead of...
catch’s picture

Status: Reviewed & tested by the community » Fixed

Committed/pushed to 9.2.x and cherry-picked to 9.1.x, thanks!

Status: Fixed » Closed (fixed)

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