Project

General

Profile

Bug #9648

localization does not work on Debian based distributions

Added by Anonymous over 7 years ago. Updated over 7 years ago.

Status:
Closed
Priority:
Low
Assignee:
Category:
Hammer core
Target version:
-
Difficulty:
Triaged:
No
Bugzilla link:
Team Backlog:
Fixed in Releases:
Found in Releases:
In Kanboard:

Description

On Debian/wheezy (and Ubuntu/trusty according to Dominic) the interface is always English, while working fine according to the locale settings on FreeBSD (I presume also on RH/CentOS). Might or might not be related to #7262.


Related issues

Related to Hammer CLI - Bug #7262: Messages are not localizedClosed2014-08-26

Associated revisions

Revision 3fdbb109 (diff)
Added by Dominic Cleal over 7 years ago

fixes #9648 - only initialise text domains that have files in the dir

6786f4cf changed the domain to avoid configuring two identical text domains,
but this reverts that and instead makes the availability check more specific
by checking for the .mo files inside the dir.

This prevents /usr/share/locale being added as a domain when the .mo files
haven't been installed there.

It also only configures the first domain that is available.

Revision 9e7ad811 (diff)
Added by Dominic Cleal over 7 years ago

refs #9648 - use correct domain for system locales, only load one domain

This reverts commit 96ee3d8cb0f0da278bac50ceb4cc15e9092e382f.

Revision 490d2ded
Added by Martin Bacovsky over 7 years ago

Merge pull request #166 from domcleal/9648-i18n-debian

fixes #9648 - only initialise text domains that have files in the dir

Revision f85e1df3
Added by mbacovsky over 7 years ago

Merge pull request #173 from domcleal/9648-i18n-debian

refs #9648 - use correct domain for system locales, only load one domain

History

#1 Updated by Dominic Cleal over 7 years ago

  • Related to Bug #7262: Messages are not localized added

#2 Updated by Dominic Cleal over 7 years ago

Yeah, reverting #7262 fixes the issue for me on 14.04. I guess this causes the wrong domain name to be added to fast_gettext then, so it would be searching for etc.

#3 Updated by Dominic Cleal over 7 years ago

  • Status changed from New to Assigned
  • Assignee set to Dominic Cleal

#4 Updated by The Foreman Bot over 7 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/hammer-cli/pull/166 added
  • Pull request deleted ()

#5 Updated by Dominic Cleal over 7 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

Also available in: Atom PDF