acquia / blt

Acquia's toolset for automating Drupal 8 and 9 development, testing, and deployment.
https://docs.acquia.com/blt/
GNU General Public License v2.0
442 stars 394 forks source link

DX-5186: Drupal 10 Codebase + New BLT Project PHP Fatal Errors #4623

Closed mikemadison13 closed 1 year ago

mikemadison13 commented 1 year ago

Describe the bug After https://github.com/acquia/blt/issues/4618 exposed issues with our new project setup, I wanted to test new project creation on PHP 8.1 on Drupal 10 to confirm if that is in fact working (and it's not).

To Reproduce Steps to reproduce the behavior, ideally starting from a fresh install of BLT:

  1. Run composer create-project acquia/drupal-recomcmended-project --no-interaction
  2. Remove acquia cms starter kit, upgrade status, etc.
  3. Up the php settings to 8.1
  4. Add BLT

Expected behavior It works

Detailed error output

11 package suggestions were added by new dependencies, use `composer suggest` to see details.
Package webmozart/path-util is abandoned, you should avoid using it. Use symfony/filesystem instead.
Generating autoload files
73 packages you are using are looking for funding.
Use the `composer fund` command to find out more!
Scaffolding files for acquia/drupal-recommended-project:
  - Skip [profile-root]/.editorconfig: disabled
  - Skip [profile-root]/.gitattributes: disabled
  - Skip [profile-root]/.travis.yml: disabled
  - Skip [profile-root]/acquia-pipelines.yml: disabled
  - Skip [web-root]/sites/default/default.services.yml because it already exists and overwrite is false.
  - Skip [web-root]/sites/default/default.settings.php because it already exists and overwrite is false.
Creating BLT template files...
 > /Users/mike.madison/git/examples/drupal10/vendor/acquia/blt/bin/blt internal:add-to-project --ansi -n
PHP Fatal error:  Declaration of Robo\Task\BaseTask::setLogger(Psr\Log\LoggerInterface $logger) must be compatible with Psr\Log\LoggerAwareInterface::setLogger(Psr\Log\LoggerInterface $logger): void in /Users/mike.madison/git/examples/drupal10/vendor/consolidation/robo/src/Common/TaskIO.php on line 30

Fatal error: Declaration of Robo\Task\BaseTask::setLogger(Psr\Log\LoggerInterface $logger) must be compatible with Psr\Log\LoggerAwareInterface::setLogger(Psr\Log\LoggerInterface $logger): void in /Users/mike.madison/git/examples/drupal10/vendor/consolidation/robo/src/Common/TaskIO.php on line 30
BLT installation failed! Please execute /Users/mike.madison/git/examples/drupal10/vendor/acquia/blt/bin/blt internal:add-to-project --ansi -n --verbose to debug the issue.

In Plugin.php line 165:

  Installation aborted due to error  
mikemadison13 commented 1 year ago

cross posted on https://github.com/consolidation/robo/issues/1127