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

applets/clock: Fix compatibility with musl libc #1452

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sgn
Copy link

@sgn sgn commented Oct 5, 2024

%l is an glibc extension for strftime, this extensions isn't supported by musl libc.

Replace %l with %_I, which is functionaly equivalence. %_I itself is also an extensions, however, %_I is supported by both glibc, musl, FreeBSD and deriviates, AIX, and Solaris.

Fix: #1451

@lukefromdc
Copy link
Member

I cannot test this except for breaking changes with glibc, we don't have musl libc on Debian Unstable

@sgn
Copy link
Author

sgn commented Oct 13, 2024

Does it break for glibc? We have musl over here, and it seems to work correctly.

@lukefromdc
Copy link
Member

lukefromdc commented Oct 13, 2024 via email

%l is an glibc extension for strftime, this extensions isn't supported
by musl libc.

Replace %l with %_I, which is functionaly equivalence.
%_I itself is also an extensions, however, %_I is supported by both
glibc, musl, FreeBSD and deriviates, AIX, and Solaris.

Fix: mate-desktop#1451
Copy link
Member

@lukefromdc lukefromdc left a comment

Choose a reason for hiding this comment

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

Didn't get any problems with this using glibc in Debian Unstable, so we seem to be good on that end

@lukefromdc lukefromdc requested a review from a team October 15, 2024 19:21
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.

Clock applet: 12-hour time format broken with musl libc
2 participants