Labels

Comments

@NicolasGuilloux

Hi there !

There is a deprecated for Symfony version above 4.2 where the Process requires an array when built. I noticed this thing for the Async Command bundle but may be present elsewhere. A quick fix is to check the version of Symfony, and choose the appropriate function like bellow:

if (Kernel::MAJOR_VERSION >= 4 && Kernel::MINOR_VERSION >= 2) {
    $process = Process::fromShellCommandline($phpBin.' '.$consoleBin.' '.$this->getCommandLine($command), $this->projectDir);
} else {
    $process = new Process($phpBin.' '.$consoleBin.' '.$this->getCommandLine($command), $this->projectDir);
}
@Steveb-p

It's a deprecation. Process constructor will fall back to fromShellCommandline until Symfony 5.0.

Prefered solution would be to use an array argument.

@stale

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stalestale bot added the wontfix label Nov 21, 2019
@stalestale bot removed the wontfix label Nov 21, 2019
Sign up for free to join this conversation on . Already have an account? Sign in to comment
None yet

No branches or pull requests