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 support for compiling on Ubuntu 22.04 #217

Closed
wants to merge 4 commits into from

Conversation

progval
Copy link

@progval progval commented Dec 6, 2022

Hi,

I noticed build is broken on Ubuntu 22.04. This PR adds CI for this version in addition to 20.04 (GitHub is currently moving ubuntu-latestfrom 20.04 to 22.04, but this repo was not migrated yet).

There is an ugly fix that seems to be enough to fix compilation with OpenSSL 3 (#182), but now it seems there are a bunch of link issues with glibc:

2022-12-06T19:40:07.2675814Z /usr/bin/ld: res.o: in function `query_name':
2022-12-06T19:40:07.2676638Z /home/runner/work/bahamut/bahamut/src/res.c:622: undefined reference to `__res_mkquery'
2022-12-06T19:40:07.2677194Z /usr/bin/ld: res.o: in function `proc_answer':
2022-12-06T19:40:07.2839313Z /home/runner/work/bahamut/bahamut/src/res.c:870: undefined reference to `__dn_expand'
2022-12-06T19:40:07.2840548Z /usr/bin/ld: /home/runner/work/bahamut/bahamut/src/res.c:968: undefined reference to `__dn_expand'
2022-12-06T19:40:07.2841219Z make[1]: Leaving directory '/home/runner/work/bahamut/bahamut/src'
2022-12-06T19:40:07.2842219Z /usr/bin/ld: /home/runner/work/bahamut/bahamut/src/res.c:1138: undefined reference to `__dn_expand'
2022-12-06T19:40:07.2842517Z Building doc
2022-12-06T19:40:07.2843444Z /usr/bin/ld: /home/runner/work/bahamut/bahamut/src/res.c:1270: undefined reference to `__dn_expand'

and I can't figure how to resolve them

Feel free to pick up from this.

@progval progval marked this pull request as draft December 6, 2022 19:44
@progval
Copy link
Author

progval commented Jun 25, 2023

closing in favor of #219

@progval progval closed this Jun 25, 2023
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.

1 participant