-
Notifications
You must be signed in to change notification settings - Fork 7.1k
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
Fix statx on ARM #47008
Fix statx on ARM #47008
Conversation
Still verifying if the fix helps. I also consider a binary compatibility check for ARM, similar to the existing one. |
Verified, works. Will add the compat check for ARM in a separate PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool! I hope the struct layout matches.
Yes, the struct layout is the same across different platforms. InstallPackagesTestRelease:
ClickHouse Stateless Tests (ubsan) [2/2]:
|
Backport #47008 to 22.12: Fix statx on ARM
Backport #47008 to 23.2: Fix statx on ARM
Backport #47008 to 23.1: Fix statx on ARM
Fixes #46055
Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Fix error during server startup on old distros (e.g. Amazon Linux 2) and on ARM that glibc 2.28 symbols are not found