Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork9.7k
[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 ourterms of service andprivacy 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
carsonbot commentedOct 24, 2025
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 |
xabbuh commentedOct 24, 2025
FTR, we bumped the requirement for Redis in the Is having support for Redis < 6.2 something essential? |
alexander-schranz commentedOct 24, 2025
@xabbuh so |
xabbuh commentedOct 24, 2025
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. |
alexander-schranz commentedOct 24, 2025 • 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.
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 |
alexander-schranz commentedOct 24, 2025
@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 |
xabbuh commentedOct 24, 2025
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. |
alexander-schranz commentedOct 24, 2025
@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.