Opened 12 years ago

Closed 11 years ago

Last modified 8 years ago

#1441 closed defect (fixed)

move ~/.i18n parsing from sugar-emulator to sugar

Reported by: sascha_silbe Owned by: sascha_silbe
Priority: Unspecified by Maintainer Milestone:
Component: Sugar Version: Git as of bugdate
Severity: Minor Keywords: r+
Cc: sayamindu Distribution/OS: Other (name your distribution in the description)
Bug Status: Assigned

Description

The control panel stores locale settings in ~/.i18n, but currently this file
is only read by Fedora scripts and sugar-emulator. Parsing it in sugar will
make language selection work in "native" sessions on all distros.

Attachments (1)

1441.patch (2.1 KB) - added by sascha_silbe 12 years ago.
move ~/.i18n parsing from sugar-emulator to sugar

Download all attachments as: .zip

Change History (7)

Changed 12 years ago by sascha_silbe

move ~/.i18n parsing from sugar-emulator to sugar

comment:1 Changed 11 years ago by tomeu

  • Cc sayamindu added

Sayamindu, could you please comment? I'm a bit confused at why we can remove that code now.

comment:2 Changed 11 years ago by tomeu

(07:36:50 PM) unmadindu: tomeu: I think it makes sense, since AFAIK, ~/.i18n is Fedora specific (it is read by one of the scripts in /etc/profile.d)
(07:37:51 PM) tomeu: unmadindu: so we went that way just because of fedora inertia?
(07:38:22 PM) unmadindu: I think it is some code left over from the day when Sugar only used to run on Fedora

comment:3 Changed 11 years ago by tomeu

  • Keywords r+ added; r? removed

Thanks, please push.

comment:4 Changed 11 years ago by sascha_silbe

  • Resolution set to fixed
  • Status changed from new to closed

Pushed as [git.sugarlabs.org/projects/sugar-toolkit/repos/mainline/commits/727ea38d0bc494840a0a1d8eb464b7e0e6cb73ba 727ea38], thanks!

comment:5 Changed 11 years ago by sascha_silbe

727ea38 is the correct link, of course.

comment:6 Changed 8 years ago by dnarvaez

  • Milestone 0.88 deleted

Milestone 0.88 deleted

Note: See TracTickets for help on using tickets.