Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

[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

Conversation

@alexander-schranz
Copy link
Contributor

QA
Branch?7.4
Bug fix?no
New feature?no
Deprecations?no
IssuesFix #...
LicenseMIT

Check if 6.0 would do also the Job.

@carsonbot
Copy link

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
Copy link
Member

FTR, we bumped the requirement for Redis in the7.4 branch to get rid of things likehttps://github.com/symfony/symfony/blob/7.3/src/Symfony/Component/Cache/Traits/Redis6ProxyTrait.php.

Is having support for Redis < 6.2 something essential?

@alexander-schranz
Copy link
ContributorAuthor

@xabbuh so^6.1 would work? I'm struggling getting 6.2 on my systems not sure it always go on6.1. Currently skipped it via--ignore-platform-reqs but thats not a solution for all.

@xabbuh
Copy link
Member

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
Copy link
ContributorAuthor

alexander-schranz commentedOct 24, 2025
edited
Loading

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:

rm -rf /private/tmp/pear/temp/redispecl uninstall redis# remove `extension="redis.so" from php.inipecl install redismv /opt/homebrew/Cellar/php/8.4.6/pecl/20240924/redis.so /opt/homebrew/lib/php/pecl/20240924/redis.so# readd `extension="redis.so" to php.ini if not automatically than

SadlyPIE didn't even build which I need to recheck what was going wrong there.

@alexander-schranz
Copy link
ContributorAuthor

@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 on6.1.0-3:https://packages.ubuntu.com/search?keywords=php8.4-redis&searchon=names&suite=plucky&section=all

@xabbuh
Copy link
Member

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
Copy link
ContributorAuthor

@xabbuh I created PR here:#62149 I temporary adopted the extension install so the tests using6.1.0.

I could not adopt the windows test as there is no binary utils for redis 6.1:https://github.com/symfony/binary-utils/releases

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@alexander-schranz@carsonbot@xabbuh

[8]ページ先頭

©2009-2025 Movatter.jp