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

fix use of deprecated c-ares api (backport #15837) #15937

Merged
merged 2 commits into from
May 6, 2024

Conversation

mergify[bot]
Copy link

@mergify mergify bot commented May 6, 2024

The c-ares api has deprecated ares_process() and ares_gethostbyname() and it causes a build failure on Fedora 40.

This PR replaces the deprecated functions with their replacements.


This is an automatic backport of pull request #15837 done by Mergify.

acooks added 2 commits May 6, 2024 07:12
ares_process(...) has been deprecated.
Replace it with ares_process_fd(...)

Signed-off-by: Andrew Cooks <acooks.at.bda@gmail.com>
(cherry picked from commit 4540fa0)
c-ares has deprecated ares_gethostbyname() in version 1.28.0
Replace it with ares_getaddrinfo().

This fixes a build error on Fedora 40.

Signed-off-by: Andrew Cooks <acooks.at.bda@gmail.com>
(cherry picked from commit 89a2e4d)
@frrbot frrbot bot added the libfrr label May 6, 2024
@donaldsharp donaldsharp merged commit 6ff958a into stable/9.1 May 6, 2024
11 of 12 checks passed
@mergify mergify bot deleted the mergify/bp/stable/9.1/pr-15837 branch May 6, 2024 14:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants