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 648840 - View > Show Text _Cursor menu item appears untranslated
View > Show Text _Cursor menu item appears untranslated
Status: RESOLVED FIXED
Product: yelp
Classification: Applications
Component: General
3.0.x
Other Linux
: Normal normal
: ---
Assigned To: Yelp maintainers
Yelp maintainers
Depends on:
Blocks:
 
 
Reported: 2011-04-28 06:38 UTC by David Planella
Modified: 2011-04-29 13:59 UTC
See Also:
GNOME target: ---
GNOME version: 2.91/3.0


Attachments
patch (568 bytes, patch)
2011-04-28 11:39 UTC, Matthias Clasen
none Details | Review
Fix the build (587 bytes, patch)
2011-04-29 08:38 UTC, Gabor Kelemen
none Details | Review

Description David Planella 2011-04-28 06:38:10 UTC
(Forwarded from the downstream bug at https://bugs.launchpad.net/ubuntu/+source/yelp/+bug/772178)

When starting yelp and opening the View menu, the "Show Text _Cursor" menu entry appears always in English, regardless of the locale and regardless of the fact that it has been translated in the PO file.
Comment 1 Matthias Clasen 2011-04-28 11:15:55 UTC
Same for 'Read Link Later'. Those are all strings that are translated in the code with _(). The problem is that yelp does not set up translations at all anymore, so you only get translations for those strings where GETTEXT_DOMAIN is somehow passed to gtk, e.g. in .ui files or via gtk_action_group_set_domain.

Yelp still needs to call setlocale, bindtextdomain, etc somewhere early on.
Comment 2 Matthias Clasen 2011-04-28 11:39:56 UTC
Created attachment 186812 [details] [review]
patch
Comment 3 Shaun McCance 2011-04-28 13:35:33 UTC
Thanks for the patch, Matthias.
Comment 4 Gabor Kelemen 2011-04-29 08:38:06 UTC
I'm getting this error when building from jhbuild:

make[2]: Entering directory `/home/gabor/checkout/gnome/yelp/src'
  CC     yelp-yelp-application.o
  CC     yelp-yelp-window.o
  CC     yelp-yelp.o
yelp.c: In function ‘main’:
yelp.c:41: warning: implicit declaration of function ‘setlocale’
yelp.c:41: warning: nested extern declaration of ‘setlocale’
yelp.c:41: error: ‘LC_ALL’ undeclared (first use in this function)
yelp.c:41: error: (Each undeclared identifier is reported only once
yelp.c:41: error: for each function it appears in.)
make[2]: *** [yelp-yelp.o] Error 1
Comment 5 Gabor Kelemen 2011-04-29 08:38:57 UTC
Created attachment 186859 [details] [review]
Fix the build
Comment 6 Shaun McCance 2011-04-29 13:59:18 UTC
Thanks, Gabor. I pushed that to master.