Skip to content
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

Migrate to pcre2 #2

Closed
wants to merge 1 commit into from
Closed

Migrate to pcre2 #2

wants to merge 1 commit into from

Conversation

heitbaum
Copy link
Contributor

Update to PCRE2 that replaces the API in the original PCRE
library, which is now obsolete and no longer maintained.

Signed-off-by: Rudi Heitbaum rudi@heitbaum.com

Update to PCRE2 that replaces the API in the original PCRE
library, which is now obsolete and no longer maintained.

Signed-off-by: Rudi Heitbaum <rudi@heitbaum.com>
@heitbaum
Copy link
Contributor Author

heitbaum commented Aug 14, 2022

@tomjfr
Copy link
Collaborator

tomjfr commented Aug 20, 2022

pcre.org does not contribute a libpcre2.pc. I would suggest to test pkg-config --exists libpcre2-posix and to keep a fallback to libpcre.pc for systems without pcre2.
As epgsearch uses only extended regular expressions, it shouldn't matter anyhow - but with the suggested changes it will be future-proof

@tomjfr
Copy link
Collaborator

tomjfr commented Sep 22, 2022

There was no updated commit, so I can't accept the pull request

@tomjfr tomjfr closed this Sep 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants