Bug 1250238 - Wrong C.UTF-8 locale
Summary: Wrong C.UTF-8 locale
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: plasma-desktop
Version: 22
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Rex Dieter
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 902094
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-08-04 19:30 UTC by Ivan Romanov
Modified: 2015-09-18 12:28 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-09-18 12:28:45 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
Patch to use C locale instead of C.UTF-8 (742 bytes, patch)
2015-08-04 19:30 UTC, Ivan Romanov
no flags Details | Diff


Links
System ID Private Priority Status Summary Last Updated
KDE Software Compilation 348678 0 NOR RESOLVED Invalid locale being set 2020-11-12 17:26:13 UTC

Description Ivan Romanov 2015-08-04 19:30:33 UTC
Created attachment 1059208 [details]
Patch to use C locale instead of C.UTF-8

Regional Settings uses C.UTF-8 locale for Default -  (C). But this is not supported in Fedora yet. 

I wrote a patch to use C instead of C.UTF-8. It works for me. I think the patch is not for upstream. It for Fedora only and must be dropped when Bug 902094 will be closed.

Comment 1 Rex Dieter 2015-08-21 16:22:22 UTC
Asked onlist for feedback,
https://lists.fedoraproject.org/pipermail/kde/2015-August/015615.html

Comment 2 Daniel Vrátil 2015-08-21 23:38:38 UTC
The patch looks good to me. Minor nitpick is that it should be

  cvalue != QLatin1String("C") 

to compile with QT_NO_CAST_FROM_ASCII) or

  locale.language() != QLocale::C

to avoid string comparision completely.

Comment 3 Ivan Romanov 2015-09-08 12:50:33 UTC
Any news here?

Comment 4 Rex Dieter 2015-09-09 13:37:05 UTC
I'm importing the patch today, using 
locale.language() != QLocale::C
variant as suggested in comment #2

Comment 5 Rex Dieter 2015-09-09 13:39:13 UTC
%changelog
* Wed Sep 09 2015 Rex Dieter <rdieter> 5.4.0-4
- Wrong C.UTF-8 locale (#1250238)

Comment 6 Ivan Romanov 2015-09-09 15:36:59 UTC
Thanks!

Comment 7 Ivan Romanov 2015-09-11 13:19:47 UTC
Rex look at this comment https://bugs.kde.org/show_bug.cgi?id=348678#c4

Comment 8 Rex Dieter 2015-09-11 13:37:39 UTC
I think he's wrong, I'm not aware of any downstream customizations around locale handling.... but I'm not a glibc expert either.

Comment 9 Ivan Romanov 2015-09-18 06:58:42 UTC
Rex, seems Fedora now has C.UTF-8 and the patch can be dropped.
https://bugzilla.redhat.com/show_bug.cgi?id=902094#c20

Comment 10 Rex Dieter 2015-09-18 12:22:50 UTC
Looks like it was added only for f24+'s glibc, as far as I can tell, so we'll probably want to keep this for f22/f23.


Note You need to log in before you can comment on or make changes to this bug.