Skip to content

Avoid incompatibility with symfony/console 7#52923

Merged
fabpot merged 1 commit intosymfony:5.4from
jdecool:fix-console-conflict
Dec 8, 2023
Merged

Avoid incompatibility with symfony/console 7#52923
fabpot merged 1 commit intosymfony:5.4from
jdecool:fix-console-conflict

Conversation

@jdecool
Copy link
Copy Markdown
Contributor

@jdecool jdecool commented Dec 7, 2023 •
edited
Loading

Q A
Branch? 5.4
Bug fix? yes
New feature? no
Deprecations? no
Issues -
License MIT

The FrameworkBundle has no constraint on the symfony/console component.

So a project can have a composer.json similar to this one:

{
    "require": {
        "php": ">=8.1",
        "ext-ctype": "*",
        "ext-iconv": "*",
        "symfony/console": "^7.0",
        "symfony/framework-bundle": "5.4.*"
    },
}

Using this configuration, it will install the console component in version 7.0.1 and the framework in 5.4.32.

But this error will occur:

!!
!!  Fatal error: Declaration of Symfony\Bundle\FrameworkBundle\Console\Application::doRun(Symfony\Component\Console\Input\InputInterface $input, Symfony\Component\Console\Output\OutputInterface $output) must be compatible with Symfony\Component\Console\Application::doRun(Symfony\Component\Console\Input\InputInterface $input, Symfony\Component\Console\Output\OutputInterface $output): int in /Users/jeremy/Workspace/personnal/bug-test/vendor/symfony/framework-bundle/Console/Application.php on line 72
!!  PHP Fatal error:  Declaration of Symfony\Bundle\FrameworkBundle\Console\Application::doRun(Symfony\Component\Console\Input\InputInterface $input, Symfony\Component\Console\Output\OutputInterface $output) must be compatible with Symfony\Component\Console\Application::doRun(Symfony\Component\Console\Input\InputInterface $input, Symfony\Component\Console\Output\OutputInterface $output): int in /Users/jeremy/Workspace/personnal/bug-test/vendor/symfony/framework-bundle/Console/Application.php on line 72
!!  Symfony\Component\ErrorHandler\Error\FatalError {#16
!!    #message: "Compile Error: Declaration of Symfony\Bundle\FrameworkBundle\Console\Application::doRun(Symfony\Component\Console\Input\InputInterface $input, Symfony\Component\Console\Output\OutputInterface $output) must be compatible with Symfony\Component\Console\Application::doRun(Symfony\Component\Console\Input\InputInterface $input, Symfony\Component\Console\Output\OutputInterface $output): int"
!!    #code: 0
!!    #file: "./vendor/symfony/framework-bundle/Console/Application.php"
!!    #line: 72
!!    -error: array:4 [
!!      "type" => 64
!!      "message" => "Declaration of Symfony\Bundle\FrameworkBundle\Console\Application::doRun(Symfony\Component\Console\Input\InputInterface $input, Symfony\Component\Console\Output\OutputInterface $output) must be compatible with Symfony\Component\Console\Application::doRun(Symfony\Component\Console\Input\InputInterface $input, Symfony\Component\Console\Output\OutputInterface $output): int"
!!      "file" => "/Users/jeremy/Workspace/personnal/bug-test/vendor/symfony/framework-bundle/Console/Application.php"
!!      "line" => 72
!!    ]
!!  }
!!

So I suggest to add a conflict to avoid this issue.

@carsonbot carsonbot added this to the 5.4 milestone Dec 7, 2023
@jdecool jdecool changed the title Avoid incompatibility with symfony/console 7 [FrameworkBundle] Avoid incompatibility with symfony/console 7 Dec 7, 2023
@jdecool jdecool mentioned this pull request Dec 7, 2023
Merged
@carsonbot carsonbot changed the title [FrameworkBundle] Avoid incompatibility with symfony/console 7 Avoid incompatibility with symfony/console 7 Dec 8, 2023
@fabpot
Copy link
Copy Markdown
Member

fabpot commented Dec 8, 2023

Thank you @jdecool.

@fabpot fabpot merged commit a3361bd into symfony:5.4 Dec 8, 2023
@jdecool jdecool deleted the fix-console-conflict branch December 8, 2023 14:13
@xabbuh
Copy link
Copy Markdown
Member

xabbuh commented Dec 8, 2023

Not sure how helpful this change actually is as Composer could simply stick with FrameworkBundle 5.4.32 or lower and still install version 7 of the Console component.

This was referenced Dec 30, 2023
Merged
Merged
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@fabpot fabpot fabpot approved these changes

@stof stof stof approved these changes

@nicolas-grekas nicolas-grekas Awaiting requested review from nicolas-grekas

Assignees

No one assigned

Labels

Bug Status: Reviewed

Projects

None yet

Milestone

5.4

Development

Successfully merging this pull request may close these issues.

5 participants

@jdecool @fabpot @xabbuh @stof @carsonbot