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 382495 - orca --setup is not localized
orca --setup is not localized
Status: RESOLVED FIXED
Product: orca
Classification: Applications
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: Rich Burridge
Orca Maintainers
Depends on:
Blocks:
 
 
Reported: 2006-12-05 06:09 UTC by Takao Fujiwara
Modified: 2006-12-06 13:45 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch for src/orca/orca_i18n.py.in (609 bytes, patch)
2006-12-05 06:12 UTC, Takao Fujiwara
none Details | Review
Reworked patch. (737 bytes, patch)
2006-12-05 20:45 UTC, Rich Burridge
none Details | Review

Description Takao Fujiwara 2006-12-05 06:09:08 UTC
orca needs to define textdomain for glade.

To reproduce:
1. Invoke orca --setup

the window pop up successfully ,but it is wholely unlocalized.
I'm attaching the patch.
Comment 1 Takao Fujiwara 2006-12-05 06:12:13 UTC
Created attachment 77707 [details] [review]
Patch for src/orca/orca_i18n.py.in

Attached the patch.
Comment 2 Rich Burridge 2006-12-05 19:45:15 UTC
Thankyou! Patch checked in CVS HEAD. Closing bug as FIXED.
This just missed the Orca 2.17.3 tarball release on Sunday, but will
appear in the 2.17.4 release in about 2-3 weeks time.
Comment 3 Rich Burridge 2006-12-05 20:45:21 UTC
Created attachment 77756 [details] [review]
Reworked patch.

On trying this with a supposedly running version of Orca (i.e. one
that didn't require --setup), Will found that the original patch 
didn't work (gtk.glade wasn't loaded at that time).

I've reworked the patch and checked it into CVS HEAD. It works for me.
Will, if it still doesn't work for you, please let me know. Thanks.
Comment 4 Takao Fujiwara 2006-12-06 09:07:24 UTC
Your change is also ok however I guess your failure was caused by missing 'import gtk.glade' below.

% cvs diff -r1.3 -r1.4 src/orca/orca_i18n.py.in
Index: src/orca/orca_i18n.py.in
===================================================================
RCS file: /cvs/gnome/orca/src/orca/orca_i18n.py.in,v
retrieving revision 1.3
retrieving revision 1.4
diff -r1.3 -r1.4
37a38,39
> gtk.glade.bindtextdomain ("@GETTEXT_PACKAGE@", localedir)
> gtk.glade.textdomain("orca")

I think the try & except are not needed. Please double check my original patch.
Thanks.
Comment 5 Willie Walker 2006-12-06 12:30:56 UTC
> I think the try & except are not needed. Please double check my original patch.

There are times where the user runs the orca command w/o the DISPLAY being set.  A common example includes configuring orca for the first time by running it from a virtual console using brltty.  In these cases, an import of gtk{.glade} will cause warnings or other things to be presented on the console.  As such, we wrap the import of gtk in try/except statements to prevent the user from getting these warnings. 

Comment 6 Takao Fujiwara 2006-12-06 13:45:36 UTC
You're right. Thanks for your explanation.