-
Notifications
You must be signed in to change notification settings - Fork 8k
Downgrade pcre2 to 10.44 #20380
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
Downgrade pcre2 to 10.44 #20380
Conversation
| caches across differing environments). (ilutov) | ||
|
|
||
| - PCRE: | ||
| . Downgrade back to PCRE2 10.44, see GH-20341. (nielsdos) |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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
DanielEScherzer
left a comment
There was a problem hiding this 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
No description provided.