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

bootctl: avoid using __WORDSIZE macro #26412

Merged
merged 1 commit into from Feb 15, 2023
Merged

Conversation

floppym
Copy link
Contributor

@floppym floppym commented Feb 14, 2023

__WORDSIZE does not seem to be documented anywhere, and is probably meant to be used internally by glibc headers.

In systemd, it was only being used in warning messages. We can avoid using it by rewording the messages slightly.

Fixes a build error with musl libc.

Bug: https://bugs.gentoo.org/894430

__WORDSIZE does not seem to be documented anywhere, and is probably
meant to be used internally by glibc headers.

In systemd, it was only being used in warning messages. We can avoid
using it by rewording the messages slightly.

Fixes a build error with musl libc.

Bug: https://bugs.gentoo.org/894430
@github-actions github-actions bot added sd-boot/sd-stub/bootctl please-review PR is ready for (re-)review by a maintainer labels Feb 14, 2023
@github-actions
Copy link

An -rc1 tag has been created and a release is being prepared, so please note that PRs introducing new features and APIs will be held back until the new version has been released.

@yuwata yuwata added good-to-merge/waiting-for-ci 👍 PR is good to merge, but CI hasn't passed at time of review. Please merge if you see CI has passed and removed please-review PR is ready for (re-)review by a maintainer labels Feb 15, 2023
@yuwata
Copy link
Member

yuwata commented Feb 15, 2023

This just updates log messages. Hence, should be ok to merge even after -rc3.

@yuwata yuwata merged commit 7cd137e into systemd:main Feb 15, 2023
@github-actions github-actions bot removed the good-to-merge/waiting-for-ci 👍 PR is good to merge, but CI hasn't passed at time of review. Please merge if you see CI has passed label Feb 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

2 participants