Skip to content

Conversation

@thekhalifa
Copy link
Contributor

libpcre (a.k.a pcre3) is obsolete in favour of pcre2 by upstream. While pcre is still available, it is being removed from several linux distributions [1] [2]

  • Change CRegex to use the new version and remove inclusion of the old pcre library.
  • Remove inclib "pcre" from fbchkdoc. Note: not replaced with pcre2 here as this should be included from the pcre2.bi header

Tested on linux with only libpcre2 installed

  1. https://fedoraproject.org/wiki/Changes/PcreDeprecation
  2. https://lists.debian.org/debian-devel/2025/01/msg00621.html

libpcre (a.k.a pcre3) is obsolete in favour of pcre2 by upstream.
While pcre is still available, it is being removed from several
linux distributions [1] [2]

- Change CRegex to use the new version and remove inclusion
  of the old pcre library.
- Remove inclib "pcre" from fbchkdoc. Note: not replaced with
  pcre2 here as this should be included from the pcre2.bi header

Tested on linux with only libpcre2 installed

1. https://fedoraproject.org/wiki/Changes/PcreDeprecation
2. https://lists.debian.org/debian-devel/2025/01/msg00621.html
After changing doc/libfbdoc to pcre2, the travis build
fails due to missing library
@jayrm
Copy link
Member

jayrm commented Feb 18, 2025

@thekhalifa

After fixing CRegex.bas:111 no more compile warnings. Tested on windows and works ok for me to build documentation.

If you could fix that up that would be great, then I'll merge in. Thanks.

- fix typo in PR: free compile context
- Change CRegex.bas to use pcre2 since the libpcre (a.k.a pcre3)
  is obsolete in favour of pcre2 by upstream (Ahmad Khalifa)
- update changelog.txt
@jayrm
Copy link
Member

jayrm commented Feb 18, 2025

I am feeling impatient today, I will make the edit and update the changelog.txt. Thanks for the PR.

@jayrm jayrm merged commit 4e5f625 into freebasic:master Feb 18, 2025
@thekhalifa
Copy link
Contributor Author

Well spotted, yes, that was definitely meant to _free() not _create(). And thanks.

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