FS#73982 - [waybar] 0.9.9-2 breaks date format characters like %l and %P (regression from 0.9.9-1)

Attached to Project: Community Packages
Opened by Chris Morgan (ChrisMorgan) - Monday, 28 February 2022, 02:12 GMT
Last edited by Brett Cornwall (ainola) - Wednesday, 11 May 2022, 19:07 GMT
Task Type Bug Report
Category Packages
Status Closed
Assigned To Brett Cornwall (ainola)
Maxim Baz (maximbaz)
Architecture All
Severity Medium
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:
As of waybar 0.9.9-2, clock widgets are broken when their format string includes %l (hour, space padded, 1..12) or %P (am/pm).

Minimal config file:

{"modules-left": ["clock"], "clock": {"format": "{:%Y-%m-%d %l:%M:%S %P}"}}

Steps to reproduce:

1. Install waybar 0.9.9-2 and run `waybar --config ‹config-file-path›`.
2. Note that the time is missing in the bar, and that the console shows “[error] : invalid format”.
3. Remove %l (changing to the supposedly-equivalent %_I doesn’t help), and change %P (am/pm) to %p (AM/PM) or remove it: it works.
4. To demonstrate that it is a regression, downgrade to waybar 0.9.9-1 and try again with the config file that failed in 0.9.9-2: it works.

I haven’t filed an issue on Waybar at this time since the regression is occurring in Arch package revisions.
This task depends upon

Closed by  Brett Cornwall (ainola)
Wednesday, 11 May 2022, 19:07 GMT
Reason for closing:  Upstream
Additional comments about closing:  This is unfortunately not within our control: upstream relies on libfmt, which seems to have intentionally removed support. Please voice your displeasure there so it can get fixed!
Comment by Maxim Baz (maximbaz) - Tuesday, 01 March 2022, 23:08 GMT
Thanks, I reproduce, but it was rebuild without changes to PKGBUILD other than pkgrel, and I reproduce it when building Waybar from sources, so I think the root cause is in some dependency that was updated, which made Waybar incompatible with it, so Waybar must adapt their code to fix the format string support. This seems somewhat relevant: https://github.com/Alexays/Waybar/issues/1447

Loading...