Ticket #1441 (closed defect: fixed)

Opened 4 years ago

Last modified 3 years ago

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

Reported by: sascha_silbe Owned by: sascha_silbe
Priority: Unspecified by Maintainer Milestone: 0.88
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

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

Change History

Changed 4 years ago by sascha_silbe

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

Changed 3 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.

Changed 3 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

Changed 3 years ago by tomeu

  • keywords r+ added; r? removed

Thanks, please push.

Changed 3 years ago by sascha_silbe

  • status changed from new to closed
  • resolution set to fixed

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

Changed 3 years ago by sascha_silbe

 727ea38 is the correct link, of course.

Note: See TracTickets for help on using tickets.