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

locale: Avoid using glibc specific defines on musl #23050

Closed
wants to merge 1 commit into from

Conversation

kraj
Copy link
Contributor

@kraj kraj commented Dec 18, 2022

musl does not provide some glibc-only enum members e.g. _NL_ADDRESS_LANG_NAME

Signed-off-by: Khem Raj raj.khem@gmail.com

musl does not provide some glibc-only enum members e.g. _NL_ADDRESS_LANG_NAME

Signed-off-by: Khem Raj <raj.khem@gmail.com>
@vadz
Copy link
Contributor

vadz commented Dec 19, 2022

Thanks, we probably should just do this to fix the immediate problem, but I think an even better solution would be to check for _NL_ADDRESS_LANG_NAME availability in configure (and CMake).

If anybody has any plans to do it soon, please let me know, but otherwise I'll merge this one.

Thanks again!

@vadz vadz added ok to merge Was reviewed and ok'd by at least one person backport-3.2 Fix applied to master but still has to be backported to stable 3.2 and removed ok to merge Was reviewed and ok'd by at least one person labels Dec 19, 2022
@vadz vadz closed this in 1faf179 Dec 23, 2022
@vadz vadz removed the backport-3.2 Fix applied to master but still has to be backported to stable 3.2 label Jan 26, 2023
vadz pushed a commit to vadz/wxWidgets that referenced this pull request Jan 26, 2023
Avoid using glibc specific defines when using musl, which does not
provide glibc-only enum members such as _NL_ADDRESS_LANG_NAME.

Signed-off-by: Khem Raj <raj.khem@gmail.com>

See wxWidgets#23050.

(cherry picked from commit 1faf179)
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.

None yet

2 participants