After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 654182 - Set regional settings on log in
Set regional settings on log in
Status: RESOLVED FIXED
Product: gnome-settings-daemon
Classification: Core
Component: general
3.1.x
Other Linux
: Normal normal
: ---
Assigned To: gnome-settings-daemon-maint
gnome-settings-daemon-maint
Depends on:
Blocks:
 
 
Reported: 2011-07-07 17:46 UTC by Rodrigo Moya
Modified: 2011-07-08 11:27 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch to set LC_ vars at gnome-session startup (5.31 KB, application/octet-stream)
2011-07-07 17:46 UTC, Rodrigo Moya
  Details
Real patch (5.31 KB, patch)
2011-07-07 17:48 UTC, Rodrigo Moya
none Details | Review

Description Rodrigo Moya 2011-07-07 17:46:43 UTC
Created attachment 191486 [details]
Patch to set LC_ vars at gnome-session startup

With the new 'Formats' tab in the Region & Language panel in gnome-control-center, users are able to select the regional settings they want to use. Right now, the panel just sets a GSettings key, which needs to be read by someone on log in, to set the corresponding LC_* environment variables so that applications started in the session take those settings into account when displaying times, dates, numbers and other settings.

Since users running a GNOME session might be not always using GDM, gnome-session seems to be the best fit, so attached patch implements this.
Comment 1 Rodrigo Moya 2011-07-07 17:48:18 UTC
Created attachment 191487 [details] [review]
Real patch
Comment 2 Vincent Untz 2011-07-07 17:50:54 UTC
I'd prefer to have gnome-settings-daemon use the Setenv method on the org.gnome.SessionManager interface to set this early, instead of having to put the logic in gnome-session.
Comment 3 Rodrigo Moya 2011-07-08 11:27:05 UTC
Right, seems a much better solution, so just pushed this to g-s-d master