Bug 4987 - tl-select-profile may show different translations for GTK widgets and hiveconf-sourced text
Summary: tl-select-profile may show different translations for GTK widgets and hivecon...
Status: CLOSED FIXED
Alias: None
Product: ThinLinc
Classification: Unclassified
Component: Misc (show other bugs)
Version: 4.1.1
Hardware: PC Unknown
: P2 Normal
Target Milestone: 4.2.0
Assignee: Henrik Andersson
URL:
Keywords: prosaic, samuel_tester
Depends on:
Blocks:
 
Reported: 2014-02-12 10:59 CET by Karl Mikaelsson
Modified: 2014-04-16 11:02 CEST (History)
2 users (show)

See Also:
Acceptance Criteria:

Attachments
tl-select-profile with English GTK widgets, swedish text (115.75 KB, image/png)
2014-02-12 10:59 CET, Karl Mikaelsson 		Details
View All Add an attachment (proposed patch, testcase, etc.)

Description Karl Mikaelsson cendio 2014-02-12 10:59:37 CET 
Created attachment 509 [details]
tl-select-profile with English GTK widgets, swedish text

See attached screenshot. I've verified this with tl.cendio.se (4.1.1) and a nightly build (4.1.1post_4238). The problem seems to be the locale.getlocale() call in get_hive_translated_string in tl-select-profile, which defaults to returning whatever LC_CTYPE is set to. Here's the locale tl-select-profile was running with.

> # ps -ef | grep tl-selec
> derfian   9710  9709  0 10:41 ?        00:00:00 python-thinlinc /opt/thinlinc/libexec/tl-select-profile
> # tr '\0' '\n' < /proc/9710/environ  | grep -E "LC|LANG"
> LC_PAPER=sv_SE.utf8
> LC_ADDRESS=sv_SE.utf8
> LC_MONETARY=sv_SE.utf8
> LC_NUMERIC=sv_SE.utf8
> LC_TELEPHONE=sv_SE.utf8
> LC_MESSAGES=en_US.utf8
> LC_IDENTIFICATION=sv_SE.utf8
> LC_COLLATE=sv_SE.utf8
> LANG=en_US.utf8
> LC_MEASUREMENT=sv_SE.utf8
> LC_CTYPE=sv_SE.utf8
> LC_TIME=sv_SE.utf8
> LC_NAME=sv_SE.utf8
Comment 1 Henrik Andersson cendio 2014-03-10 15:02:48 CET 
Using LC_MESSAGES locale when fetching translated string from hive conf commited in r28582.
Comment 2 Samuel Mannehed cendio 2014-04-16 11:02:51 CEST 
Verified with multiple languages that it is consistent and not mixing languages. (build 4325)
Note You need to log in before you can comment on or make changes to this bug.