Skip to content

[7.0] Enable ext-redis and ext-apcu on appveyor #50471

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

Merged
merged 1 commit into from
Jun 21, 2023

Conversation

nicolas-grekas
Copy link
Member

@nicolas-grekas nicolas-grekas commented May 30, 2023

Q A
Branch? 7.0
Bug fix? no
New feature? no
Deprecations? no
Tickets -
License MIT
Doc PR -

Unfortunately, no DLLs are available from pecl for PHP 8.2 on Windows.
See:

BUT https://phpdev.toolsforresearch.com/php-8.2.6-Win32-vs16-x86.zip ships the ones we need, thanks to @Jan-E!

@nicolas-grekas
Copy link
Member Author

nicolas-grekas commented Jun 13, 2023

PR updated with php_apcu.dll from https://phpdev.toolsforresearch.com/php-8.2.6-Win32-vs16-x86.zip, thanks to @Jan-E!
php_redis.dll doesn't seem to work so I removed testing redis on appveyor. We already have it in our integration tests anyway.

@nicolas-grekas nicolas-grekas changed the title [7.0] Enable ext-redis and ext-apcu on appveyor [7.0] Enable ext-apcu on appveyor Jun 13, 2023
@Jan-E
Copy link

Jan-E commented Jun 13, 2023

Redis in my builds needs igbinary as well. If you build redis yourself on Windows, try --disable-redis-igbinary.

@nicolas-grekas
Copy link
Member Author

Oh thanks for the hint! PR updated, works!

@nicolas-grekas nicolas-grekas changed the title [7.0] Enable ext-apcu on appveyor [7.0] Enable ext-redis and ext-apcu on appveyor Jun 13, 2023
@Jan-E
Copy link

Jan-E commented Jun 13, 2023

For the failing 8.3 test: igbinary does not compile yet, but apcu and redis do work:
https://www.apachelounge.com/viewtopic.php?t=6617
Please note that I am still building with OpenSSL 1.1.1.

@Jan-E
Copy link

Jan-E commented Jun 15, 2023

BTW: my pleasure, as a Drupal developer.

@nicolas-grekas nicolas-grekas merged commit 824d6fe into symfony:7.0 Jun 21, 2023
@nicolas-grekas nicolas-grekas deleted the sf7-appveyor branch June 21, 2023 08:37
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