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

lib/gis: add support for PCRE #5159

Merged
merged 1 commit into from
Feb 21, 2025
Merged

lib/gis: add support for PCRE #5159

merged 1 commit into from
Feb 21, 2025

Conversation

nilason
Copy link
Contributor

@nilason nilason commented Feb 20, 2025

Add support for PCRE, where/when GNU Regex cannot be used.

As this is mainly (perhaps only) needed on Win systems, --without-pcre is default. If both Regex and PCRE are available on the system, Regex is used by default.

Co-authored-by: Huidae Cho <grass4u@gmail.com>
Co-authored-by: Nicklas Larsson <n_larsson@yahoo.com>
@nilason nilason added this to the 8.5.0 milestone Feb 20, 2025
@github-actions github-actions bot added C Related code is in C libraries labels Feb 20, 2025
@nilason nilason requested a review from HuidaeCho February 20, 2025 12:58
@nilason
Copy link
Contributor Author

nilason commented Feb 20, 2025

Rashad was the original author, wasn't he?

@wenzeslaus
Copy link
Member

We are not testing this in the CI with the assumption that the code will be eventually tested with CMake on Windows, right?

Also, I think it does not need to go in before #3621.

@nilason
Copy link
Contributor Author

nilason commented Feb 20, 2025

We are not testing this in the CI with the assumption that the code will be eventually tested with CMake on Windows, right?

I tested it locally, configure and build, at least.

Also, I think it does not need to go in before #3621.

Not necessarily, I'll only remove the code changes there. The rest can stay.

Probably easier than rebase/merge main on THAT one :-).

@HuidaeCho
Copy link
Member

Rashad was the original author, wasn't he?

I think so.

@nilason nilason merged commit f22b127 into OSGeo:main Feb 21, 2025
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C Related code is in C libraries
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants