Skip to content

Conversation

@dunglas
Copy link
Contributor

@dunglas dunglas commented Apr 19, 2024

What does this PR do?

It doesn't look currently possible to use ICU 75 with PHP because this version requires C++ 17.
php/php-src#14002 and #414 will fix the issue. In the meantime, we have to downgrade ICU to v74.

Checklist before merging

If your PR involves the changes mentioned below and completed the action, please tick the corresponding option.
If a modification is not involved, please skip it directly.

  • If it's a extension or dependency update, make sure adding related extensions in src/global/test-extensions.php.
  • If you changed the behavior of static-php-cli, add docs in static-php/static-php-cli-docs .
  • If you updated config/xxxx.json content, run bin/spc dev:sort-config xxx.

@dunglas
Copy link
Contributor Author

dunglas commented Apr 19, 2024

Closing in favor of #414.

@dunglas dunglas closed this Apr 19, 2024
@dunglas dunglas mentioned this pull request Apr 20, 2024
3 tasks
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.

1 participant