Page MenuHomeFreeBSD

PR241491: bin/ps: use locale-based string specifiers
ClosedPublic

Authored by yuripv on Oct 26 2019, 4:22 PM.
Tags
None
Referenced Files
F152622086: D22160.id.diff
Thu, Apr 16, 1:55 AM
Unknown Object (File)
Tue, Apr 7, 4:00 AM
Unknown Object (File)
Mar 17 2026, 6:07 AM
Unknown Object (File)
Mar 16 2026, 2:38 PM
Unknown Object (File)
Mar 16 2026, 4:11 AM
Unknown Object (File)
Mar 11 2026, 11:34 AM
Unknown Object (File)
Mar 6 2026, 4:14 AM
Unknown Object (File)
Mar 3 2026, 4:31 AM
Subscribers

Details

Summary

Use %hs instead of %s in bin/ps for it to correctly output locale specific date/time (at least).

Test Plan

Compare output of LC_ALL=uk_UA.UTF-8 ps -o lstart and LC_ALL=uk_UA.KOI8-U ps -o lstart | iconv -f koi8-u. Also reproducer from the PR.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Looks great! Thanks....

This revision is now accepted and ready to land.Oct 29 2019, 12:12 AM
This revision was automatically updated to reflect the committed changes.

The "%hs" bits look good, but why do you add the call to xo_no_setlocale()?

Thanks,
Phil

In D22160#555029, @phil wrote:

The "%hs" bits look good, but why do you add the call to xo_no_setlocale()?

Thanks,
Phil

Oh, I didn't notice it's already fixed in libxo, will remove.

In D22160#555029, @phil wrote:

The "%hs" bits look good, but why do you add the call to xo_no_setlocale()?

Thanks,
Phil

Removed in followup commit.