Fix portability regarding compat/base64, python mod cflags and c compilers like zig cc#1827
Open
michaelortmann wants to merge 5 commits intoeggheads:developfrom
Open
Fix portability regarding compat/base64, python mod cflags and c compilers like zig cc#1827michaelortmann wants to merge 5 commits intoeggheads:developfrom
zig cc#1827michaelortmann wants to merge 5 commits intoeggheads:developfrom
Conversation
zig cczig cc
Member
Author
|
Something is still not right regarding base64: Can't load modules server: /home/michael/eggdrop/modules/server.so: undefined symbol: __b64_pton |
zig cczig cc
Member
Author
|
this was symbol resolving hell. finally i fixed it by putting the include for resolv.h in one and only one place in eggdrop and fix it proper there. if configure didnt find a way to use a system base64 it will make eggdrop use its own compat functions. still eggdrop would include resolv.h for dns and stuff, and resolv.h was still defining b64() funcs to __b64() funcs in some cases / systems. the the final fix was to undef b64() funcs right after including resolv.h if HAVE_BASE64 wasnt set by configure. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Found by: michaelortmann
Patch by: michaelortmann
Fixes:
One-line summary:
Fix portability regarding compat/base64, python mod cflags and c compilers like
zig ccAdditional description (if needed):
$ CC="zig cc -Wno-deprecated-non-prototype" ./configure && make config && makeworks now :)Test cases demonstrating functionality (if applicable):
Fixes all make errors with
zig cc:1.
Before (note the underscores):
After:
2.
3.