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

Add padding on 32 bit statvfs with 64 bit offsets #73

Merged
merged 1 commit into from
Sep 30, 2024

Conversation

tatokis
Copy link
Contributor

@tatokis tatokis commented Sep 27, 2024

Effectively does what musl does, which also works for glibc.

Closes #71

@djberg96
Copy link
Owner

@tatokis Can you resolve merge conflict please?

@tatokis tatokis force-pushed the i686-64-bit-offset-fix branch from 0a51a5e to 305b6b7 Compare September 30, 2024 01:47
@tatokis
Copy link
Contributor Author

tatokis commented Sep 30, 2024

Done

Copy link
Owner

@djberg96 djberg96 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, specs passing. If Solaris ever makes a comeback we can worry about big endian systems then. ;)

@djberg96 djberg96 merged commit 7b6cdb1 into djberg96:main Sep 30, 2024
20 checks passed
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.

Unexpected stat(v)fs struct size on i686
2 participants