Page MenuHomeFreeBSD

Patch icu to use UTF-8 instead of ASCII outside C locale and ensure UTF-8 is default
ClosedPublic

Authored by rezny on Mar 24 2017, 10:59 AM.
Tags
None
Referenced Files
F81648672: D10128.diff
Fri, Apr 19, 11:04 AM
Unknown Object (File)
Sat, Apr 6, 6:13 AM
Unknown Object (File)
Sat, Apr 6, 2:30 AM
Unknown Object (File)
Feb 12 2024, 4:15 AM
Unknown Object (File)
Feb 12 2024, 4:15 AM
Unknown Object (File)
Feb 12 2024, 4:15 AM
Unknown Object (File)
Feb 12 2024, 4:03 AM
Unknown Object (File)
Feb 5 2024, 1:41 AM
Subscribers

Details

Summary

From PR 216372: Some QT5 (e.g. qTox) apps have trouble with non-ASCII filenames. Root cause: ucnv_getDefaultName() is called before setlocale() so it returns ASCII. The solution is to handle BSD the same as is done for Darwin and Linux; map ASCII to UTF-8 outside POSIX 'C' locale. The implementation uses the Darwin code path because it already matched in other respects (CP949 is remapped but nothing else, whereas Linux has special handling for more Asian locales). The PR has gone 2 months without action from office@

Test Plan

Poudriere 10.3/11.0 amd64/i386 OK, verified qTox now correctly handles files with Unicode names

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bapt added a subscriber: bapt.

please consider upstreaming that as well

This revision is now accepted and ready to land.Mar 24 2017, 12:38 PM
This revision was automatically updated to reflect the committed changes.