Skip to content

Update bundled pcre2 to 10.42 #12109

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

Closed
wants to merge 1 commit into from
Closed

Conversation

nielsdos
Copy link
Member

@nielsdos nielsdos commented Sep 2, 2023

To be clear: this new pcre2 version also includes the JIT backported fix in GH-12108.

@Girgias
Copy link
Member

Girgias commented Sep 2, 2023

Okay, here it may make more sense to have this be backported to 8.3 so that the latest release is on the newest version of PCRE.

@bukka, @ericmann, @adoy what it your opinion as RMs?

@adoy
Copy link
Member

adoy commented Sep 4, 2023

Okay, here it may make more sense to have this be backported to 8.3 so that the latest release is on the newest version of PCRE.

I think it make sense to release this as part of 8.3. It's only something internal and do not change anything from the user standpoint. @bukka @ericmann any input on this ?

@bukka
Copy link
Member

bukka commented Sep 8, 2023

Yeah I think it's fine. AFAIK most distros use system one pcre2 anyway. Think this is mostly for Windows though.

@nielsdos nielsdos closed this in c4e8f65 Sep 8, 2023
@nielsdos
Copy link
Member Author

nielsdos commented Sep 8, 2023

Thx for checking

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.

None yet

4 participants