Skip to content

Conversation

@ndossche
Copy link
Member

@ndossche ndossche commented Nov 3, 2025

No description provided.

caches across differing environments). (ilutov)

- PCRE:
. Downgrade back to PCRE2 10.44, see GH-20341. (nielsdos)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be good to have a test for the issue so any future upgrade does not break it again.

Also, did you considered backporting PCRE2Project/pcre2#841 a few lines fix?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It depends on the RM opinion whether we'll backport or go with this (once that PR is accepted).
In #20341 @DanielEScherzer suggested downgrading.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We shouldn't backport the upstream patch until it is merged upstream.
Given that RC4 is being tagged tomorrow, we should have 8.5 with a version of PCRE2 that isn't broken

Copy link
Member

@DanielEScherzer DanielEScherzer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

RM approval, technical review not performed

@ndossche ndossche merged commit f4b27bd into php:PHP-8.5 Nov 3, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PCRE2 10.45-10.47 contains a known regex character class compilation issue

4 participants