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

fixes for musl libc #266

Closed
wants to merge 2 commits into from
Closed

fixes for musl libc #266

wants to merge 2 commits into from

Conversation

lemmi
Copy link

@lemmi lemmi commented Mar 19, 2024

Two small changes are necessary to successfully build on musl libc.
I have been adding the ALLPERMS patch downstream for a couple of years now and forgot to post it here.
The uid_t is new and is needed to fix https://github.com/void-linux/void-packages/actions/runs/8342934314/job/22832155263?pr=49372#step:7:172

@shundhammer
Copy link
Owner

There is already BrokenLibc.h for just this purpose, and it's in use in several places; but a recent change for the new cache file format V2.0 with new fields for UID/GID/permissions (#265) also needed ALLPERMS in another file, and that was missing.

This should now work with commit 7162c0a which also includes that #include <sys/types.h>. Please check if it builds for you.

@lemmi
Copy link
Author

lemmi commented Mar 19, 2024

All green now :)

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.

2 participants