Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork9.7k
[Console] Runtime conflict for psr/log >= 3.0 instead of composer conflict#43884
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
c0b3a60 to7af9ba0Comparestof commentedNov 2, 2021
Well, to me, letting the solver know about the conflict is better than forcing the user to manage dependencies manually. Cases where the latest versions of packages are incompatible together need to be resolved by the user anyway to choose which requirement they want to relax, but that's only happening properly if the solver knows about the incompatibility. |
derrabus left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
I think, no matter how we deal with this issue, there will be a group of people dissatisfied with our solution. Currently, we have a well documented incompatibility that should be easy to resolve. Moving this to a runtime check feels like we're doing the job Composer should've done.
I'm not convinced we should make this change.
nicolas-grekas commentedNov 2, 2021 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Shouldn't we have a mix here? This exception in 5.3 and the conflict in 5.4? What about moving the check to the constructor though? |
derrabus commentedNov 2, 2021
Good point. If we restore the conflict in 5.4, I'm 👍
That wouldn't work because the class definition itself raises a fatal error, doesn't it? |
fabpot commentedNov 3, 2021
Thank you@fancyweb. |
This PR was merged into the 5.4 branch.Discussion----------[Console] Restore psr/log >= 3.0 conflict| Q | A| ------------- | ---| Branch? | 5.4| Bug fix? | no| New feature? | no| Deprecations? | no| Tickets |#43884 (comment)| License | MIT| Doc PR | -Commits-------5c65796 [Console] Restore psr/log >= 3.0 conflict
ConsoleLoggeris not compatible withpsr/log >= 3.0but it's an optional feature andpsr/logis an optional dependency in the Console component. Yet, having a real "composer conflict", preventssymfony/consolefrom being updated above the5.3.2version if a previous dependency has already installedpsr/log 3.0. But I'd rather have an updatedsymfony/consoleversion andpsr/log 2.0since thepsr/logchanges are just about types.