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

bti uses old pcre library #53

Open
gregoa opened this issue Nov 19, 2021 · 5 comments
Open

bti uses old pcre library #53

gregoa opened this issue Nov 19, 2021 · 5 comments

Comments

@gregoa
Copy link
Contributor

gregoa commented Nov 19, 2021

We have the following bug report in Debian:
https://bugs.debian.org/1000121

Summary: bti uses the old pcre library which is supposed to be faded out in Debian as it's EOL. The maintainer suggests to port bti to the currently maintained pcre2 library.

Cheers,
gregor

@paulrho
Copy link
Contributor

paulrho commented Nov 20, 2021 via email

@gregoa
Copy link
Contributor Author

gregoa commented Nov 20, 2021 via email

@paulrho
Copy link
Contributor

paulrho commented Nov 22, 2021

I compiled using PCRE2 (and guessing some mapping) - simple test appears to work, just need:

  • clean it up to make the most minimal change
  • see if what guesses I made make sense

Note, this is only used to pull out URLs and try to shrink them (e.g. bit.ly) - I don't normally use that function - so it will need to be tested.

@paulrho
Copy link
Contributor

paulrho commented Nov 22, 2021

The two link shortening methods (only two allowed) - will no longer work, bit.ly has now moved on to v4:

"If you go to https://dev.bitly.com/index.html, there's a big notice stating
'API V3 will be deactivated on March 1, 2020. If you are currently using V3, migrate to V4 as soon as possible to avoid a breakdown of your integrations. See all the changes in our updated documentation for API V4.'"

My suggestion is, make the code work (as though the shortening service is going to work) - test with a stub - then put the original (non working shortening script) back.

@paulrho
Copy link
Contributor

paulrho commented Nov 22, 2021

#54 - simplest change to move from pcre to pcre2

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

No branches or pull requests

2 participants