search

jhedstrom / DrupalDriver

A collection of drivers for controlling Drupal.
GNU General Public License v2.0
64 stars 97 forks source link

Fixed compatibility with Symfony\Process v4+ for Drush driver. #251

Closed AlexSkrypnyk closed 1 year ago

AlexSkrypnyk commented 2 years ago

DrushDriver uses Symfony\Process to issue CLI commands.

Process expects the first element to be an array. Since we are passing the Drush command with arguments as a string, it would be better to use Process::fromShellCommandline() instead of new Process() to make sure compatibility with different versions of Symfony\Process.

Without this change, drush steps are failing with Symfony v6

jhedstrom commented 1 year ago

I've merged #261. Thanks for this!