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

Build fails on musl #226

Closed
crabctrl opened this issue Jul 7, 2022 · 2 comments
Closed

Build fails on musl #226

crabctrl opened this issue Jul 7, 2022 · 2 comments

Comments

@crabctrl
Copy link

crabctrl commented Jul 7, 2022

Before you close this issue claiming that it's been fixed by recent commits, no. It hasn't.

The code still uses qsort_r(), which is not supported by musl, causing the build to fail. This is still an issue. I'm curious as to how the build was tested before in a way that gave the impression that it no longer failed.

@frozencemetery
Copy link
Member

The hostility isn't welcome.

musl is not something we the maintainers support (for reasons that, in order to keep this conversation civil, I won't go into). Patches are generally accepted, but it's not something we're testing. You're free to read other issues to see why they've been closed, but in #224 and #206 we received confirmation that things were working. If they're not, then it's news to us.

@floppym
Copy link
Contributor

floppym commented Aug 3, 2022

Support for qsort_r was added in musl-1.2.3.

@crabctrl crabctrl closed this as completed Aug 4, 2022
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

3 participants