Skip to content

Conversation

@ndossche
Copy link
Member

@ndossche ndossche 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.

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

ndossche 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.

4 participants