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
Unknown Object (File)
Wed, Oct 16, 2:24 AM
Unknown Object (File)
Mon, Oct 14, 7:23 PM
Unknown Object (File)
Mon, Oct 14, 11:03 AM
Unknown Object (File)
Sun, Oct 13, 5:26 AM
Unknown Object (File)
Sat, Oct 12, 10:00 AM
Unknown Object (File)
Fri, Oct 11, 2:48 AM
Unknown Object (File)
Wed, Oct 9, 3:23 AM
Unknown Object (File)
Tue, Oct 8, 3:40 PM
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.