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 786978 - The plugins notifications are displayed untranslated
The plugins notifications are displayed untranslated
Status: RESOLVED FIXED
Product: gnome-settings-daemon
Classification: Core
Component: plugins
3.25.x
Other Linux
: Normal normal
: ---
Assigned To: gnome-settings-daemon-maint
gnome-settings-daemon-maint
Depends on:
Blocks:
 
 
Reported: 2017-08-29 14:13 UTC by Sebastien Bacher
Modified: 2017-08-31 15:27 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
call setlocale so the notifications are translated (1.48 KB, patch)
2017-08-29 14:18 UTC, Sebastien Bacher
none Details | Review
call setlocale so the notifications are translated (1.48 KB, patch)
2017-08-29 14:20 UTC, Sebastien Bacher
committed Details | Review
common: include missing header files for LC_ALL (1.03 KB, patch)
2017-08-30 03:12 UTC, Jonathan Kang
committed Details | Review

Description Sebastien Bacher 2017-08-29 14:13:45 UTC
Using GNOME 3.25.90, the g-s-d plugins notifications display untranslated (the "tz has changed" for example), it seems setlocale() is not being called since the split and commit 950251e7
Comment 1 Sebastien Bacher 2017-08-29 14:18:42 UTC
Created attachment 358685 [details] [review]
call setlocale so the notifications are translated
Comment 2 Sebastien Bacher 2017-08-29 14:20:19 UTC
Created attachment 358686 [details] [review]
call setlocale so the notifications are translated

same with a better description
Comment 3 Rui Matos 2017-08-29 14:59:53 UTC
Review of attachment 358686 [details] [review]:

looks good, thanks. please push to the gnome-3-24 branch too
Comment 4 Sebastien Bacher 2017-08-29 16:02:18 UTC
Review of attachment 358686 [details] [review]:

commited as https://git.gnome.org/browse/gnome-settings-daemon/commit/?id=9fddecc4, gnome-3-24 to come
Comment 5 Sebastien Bacher 2017-08-29 19:48:32 UTC
commited to gnome-3-24 as well, https://git.gnome.org/browse/gnome-settings-daemon/commit/?h=gnome-3-24&id=d35f5edd
Comment 6 Jonathan Kang 2017-08-30 03:12:47 UTC
Created attachment 358733 [details] [review]
common: include missing header files for LC_ALL
Comment 7 Rui Matos 2017-08-30 08:56:41 UTC
Review of attachment 358733 [details] [review]:

oops, thanks!
Comment 8 Rui Matos 2017-08-30 08:57:15 UTC
Attachment 358733 [details] pushed as 1bf7e49 - common: include missing header files for LC_ALL
Comment 9 Piotr Drąg 2017-08-30 14:01:09 UTC
Could bug #782837 be related?
Comment 10 Sebastien Bacher 2017-08-31 15:27:47 UTC
the bluetooth issue is a different one but I've an idea what is it and I'm going to write a patch for it as well

did the missing include create build issues somewhere? I'm mostly curious because it builds fine here and there was previous commit that added setlocale() calls without that include, e.g
https://git.gnome.org/browse/gnome-settings-daemon/commit/?id=51d18a34

you maybe want to fix that one as well then?