Opened 8 years ago

Closed 8 years ago

#4366 closed defect (fixed)

i18n error

Reported by: cjl Owned by: danielfrancis
Priority: Normal Milestone: Unspecified
Component: GraphPlotter Version: Git as of bugdate
Severity: Major Keywords: i18n
Cc: Distribution/OS: Unspecified
Bug Status: Resolved

Description

Out POT refresh scripts are throwing this error for GraphPlotter.

####### Checking POT for Graph Plotter (master) ######

Already up-to-date.
xgettext: Non-ASCII string at gtk3modules/graph.py:470.

Please specify the source encoding through --from-code.

ERROR - xgettext failed with return code 1.

Change History (5)

comment:1 Changed 8 years ago by danielfrancis

  • Bug Status changed from Unconfirmed to New
  • Keywords i18n added
  • Milestone changed from Unspecified by Release Team to not applicable
  • Priority changed from Unspecified by Maintainer to Normal
  • Severity changed from Unspecified to Major
  • Status changed from new to assigned
  • Version changed from Unspecified to Git as of bugdate

Sugar activities encode strings in UTF-8 by default. I removed the unnecessary coding header from graph.py and I see that xgettext doesn't like it.

The non-ascii character is the Pi letter used in the radian scale.
http://git.sugarlabs.org/graph-plotter/mainline/blobs/master/gtk3modules/graph.py#line470
But it isn't a message to be translated and I didn't use a _()

The error says that you can specify the source encoding modifying the pootle helper. If you think that the pootle helper can't assume the encoding, I have no problem adding again the coding header to the file.

comment:2 Changed 8 years ago by danielfrancis

  • Bug Status changed from New to Assigned

I commited a fix[1] and added a section about this to the wiki i18n tips article[2].

Chris, please close the ticket if the pootle helper works now with my activity.

[1] http://git.sugarlabs.org/graph-plotter/mainline/commit/be38309372bacb4df7fc57c86bb334d88002cfa2

[2] http://wiki.sugarlabs.org/go/Translation_Team/i18n_Best_Practices#Specify_the_character_encoding_if_necessary

comment:3 Changed 8 years ago by cjl

daniel, thanks for the patch, I will review the error logs of the pootle pot-generator cron job tonight and close the ticket if it is all clear for graphplotter.

comment:4 Changed 8 years ago by dnarvaez

  • Milestone not applicable deleted

Milestone not applicable deleted

comment:5 Changed 8 years ago by ignacio

  • Bug Status changed from Assigned to Resolved
  • Milestone set to Unspecified
  • Resolution set to fixed
  • Status changed from assigned to closed
Note: See TracTickets for help on using tickets.