drupalorg-cli icon indicating copy to clipboard operation
drupalorg-cli copied to clipboard

Uncaught TypeError Argument 1

Open Greg-Boggs opened this issue 3 years ago • 8 comments

I cleaned out my .config folder completely and installed drupalorg cli fresh

When I run drupalorg MRN I get this error on PHP 7.4.

User error maybe?

PHP Fatal error: Uncaught TypeError: Argument 1 passed to Symfony\Component\Process\Process::__construct() must be of the type array, string given, called in /home/me/.config/composer/vendor/klaussilveira/gitter/lib/Gitter/Client.php on line 82 and defined in /home/me/.config/composer/vendor/symfony/process/Process.php:143

Greg-Boggs avatar Jun 15 '22 18:06 Greg-Boggs

Looks like that dependency doesn't support symfony/process 5 or 6.

Short term fix if you're composer required (ie not using the .phar): composer require "symfony/process:v4.4.41 as v5.0.0" That may interfer with other things - but does fix release-notes.

NickDickinsonWilde avatar Jun 15 '22 23:06 NickDickinsonWilde

++

Greg-Boggs avatar Jun 16 '22 00:06 Greg-Boggs

Same problem with phar

ivnish avatar Aug 10 '22 06:08 ivnish

@mglaman I got same error in 0.4.1. Now drupalorg mrn don't work. I tried both methods: deprecated composer global and phar. So I temporary downgraded to 0.3.3.

dillix avatar Aug 11 '22 05:08 dillix

klaussilveira/gitter probably needs to be abandoned as it still has a low symfony/process dependency. Sorry, I'll look into this.

mglaman avatar Aug 17 '22 22:08 mglaman

That's the issue with klaussilveira/gitter which requires symfony/proces ^2.8 but app installing SF 5/6

andypost avatar Aug 19 '22 00:08 andypost

@mglaman maybe https://www.drupal.org/project/drupal/issues/3172537 could help but ++ to get rid of outdated dependency

andypost avatar Aug 19 '22 00:08 andypost

++ to get rid of the outdated dependency

rmcveigh avatar Oct 13 '22 16:10 rmcveigh

I created a PR to replace gitter by czproject/git-php

chrfritsch avatar Jan 04 '23 13:01 chrfritsch

Thanks @chrfritsch - it would be great to get this one in😃

AdamPS avatar Mar 23 '23 15:03 AdamPS

@mglaman any chance of a look at #183 and a new release soon? thanks!

NickDickinsonWilde avatar Apr 17 '23 06:04 NickDickinsonWilde