🌐 AI搜索 & 代理 主页
Skip to content

Conversation

@alexander-schranz
Copy link
Contributor

Q A
Branch? 7.4
Bug fix? no
New feature? no
Deprecations? no
Issues Fix #...
License MIT

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

xabbuh commented Oct 24, 2025

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

Is having support for Redis < 6.2 something essential?

@carsonbot carsonbot closed this Oct 24, 2025
@alexander-schranz
Copy link
Contributor Author

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

@xabbuh
Copy link
Member

xabbuh commented Oct 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
Copy link
Contributor Author

alexander-schranz commented Oct 24, 2025

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/redis
pecl uninstall redis

# remove `extension="redis.so" from php.ini

pecl install redis

mv /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

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

@alexander-schranz
Copy link
Contributor Author

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

@xabbuh
Copy link
Member

xabbuh commented Oct 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
Copy link
Contributor Author

@xabbuh I created PR here: #62149 I temporary adopted the extension install so the tests using 6.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 free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants