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

Port to musl libc #241

Closed
wants to merge 2 commits into from
Closed

Port to musl libc #241

wants to merge 2 commits into from

Conversation

er2off
Copy link
Contributor

@er2off er2off commented Apr 24, 2023

Closes #72 and uses some patches from #193 for better portability between UNIXes.

Also this patch replaces so many NULLs with 0 or '\0' because musl headers redefines it and anyway it is better practice.

Moreover, due to very "cool" code from Valve, some modules comes with disabled fortify which distributes with build-base in Alpine (fortify is something like compile-time checking for memory bounds)

@nillerusr nillerusr force-pushed the master branch 2 times, most recently from dfaccca to f2fa241 Compare April 24, 2023 15:54
@er2off er2off force-pushed the musl branch 3 times, most recently from b38bccd to 1654192 Compare April 24, 2023 16:16
Closes nillerusr#72 and uses some patches from nillerusr#193 for better portability
between UNIXes.

Also this patch replaces so many NULLs with 0 or '\0' because musl
headers redefines it and anyway it is better practice.

Moreover, due to very "cool" code from Valve, some modules comes with
disabled fortify which distributes with build-base in Alpine
(fortify is something like compile-time checking for memory bounds)
@nillerusr nillerusr closed this Apr 24, 2023
@er2off er2off deleted the musl branch May 10, 2023 17:14
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.

MUSL libc support?
2 participants