Bug 1250238

Summary: Wrong C.UTF-8 locale
Product: [Fedora] Fedora Reporter: Ivan Romanov <drizt72>
Component: plasma-desktopAssignee: Rex Dieter <rdieter>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 22CC: dvratil, jgrulich, kde-sig, kevin, ltinkl, rdieter, than
Target Milestone: ---Keywords: Patch
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-09-18 12:28:45 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 902094    
Bug Blocks:    
Attachments:
Description Flags
Patch to use C locale instead of C.UTF-8 none

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.