-
Notifications
You must be signed in to change notification settings - Fork 24
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
Port to PCRE2 #52
base: master
Are you sure you want to change the base?
Port to PCRE2 #52
Conversation
Solution: port to PCRE2 Link: https://bugs.debian.org/1000118
Have you tested that this doesn't introduce regressions for the variety of usage of this tool, like those in other repositories in this org? |
No. Can you point out which ones make use of it? I do not have any gsl domain knowledge. |
But note that this is incomplete because I do not have the means to adjust and test the Windows build files. |
It is used by https://github.com/zeromq/zproject which in turn is used by a large number of other repos, all those that include a project.xml in the root directory. And yes this needs to be verified on Windows and OSX too at a minimum. |
Outside of builds/msvc, replace the pcre references.
Solution: Prevent syntax error by quoting.
Are there any references to differences between the two versions? I couldn't find any when I quickly scanned the PCRE2 github repo. |
There is no porting guide, but people linked some documentation on the bug which asks for one. |
The original PCRE library is deprecated. This is a port to the successor PCRE2.