-
-
Notifications
You must be signed in to change notification settings - Fork 9.8k
[WIP] Open Symfony Cache Ext-Redis Requirement to ^6.0 #62146
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[WIP] Open Symfony Cache Ext-Redis Requirement to ^6.0 #62146
Conversation
|
Hey! To help keep things organized, we don't allow "Draft" pull requests. Could you please click the "ready for review" button or close this PR and open a new one when you are done? Note that a pull request does not have to be "perfect" or "ready for merge" when you first open it. We just want it to be ready for a first review. Cheers! Carsonbot |
|
FTR, we bumped the requirement for Redis in the Is having support for Redis < 6.2 something essential? |
|
@xabbuh so |
|
We don't have to get rid off of all the compatibility code we currently have in Symfony 7.3 just for the sake of it. If updating from Redis 6.1 to 6.2 is not that easy I am open for restoring Redis 6.1 support. I wonder though if allowing even older releases is really necessary since the release of 6.1 happened a year ago. |
|
Atleast I found now a solution to ugprade my Redis Extensions it seems pecl did move it to the false directory and some temp file blocked rebuild: Sadly |
|
@xabbuh think we still should add support for 6.1 the current Ubuntu LTS version 25.04 which are strict about supported version seems to be on |
|
Sounds reasonable, do you want to give it a try? Otherwise I can see if I can have a look at it over the weekend. |
|
@xabbuh I created PR here: #62149 I temporary adopted the extension install so the tests using I could not adopt the windows test as there is no binary utils for redis 6.1: https://github.com/symfony/binary-utils/releases |
Check if 6.0 would do also the Job.