gtypist wont start: i18n problem

Bug #281946 reported by Dmitriy Larchenko
40
This bug affects 7 people
Affects Status Importance Assigned to Milestone
gtypist (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

Binary package hint: gtypist

gtypist quit with message "(null): i18n problem: invalid value for msgid "Y/N": Д/Н" just after I start it.

Ubuntu 8.04.1, gtypist 2.7-5, locale "ru_RU.UTF-8"

Related branches

Revision history for this message
Vadim Velikodniy (vadim-velikodniy) wrote :

The reason is improperly encoding of file /usr/share/ru.typ. It uses KOI8-R.

You can recode the file with iconv to UTF-8 or run English version of interface (LANG=C gtypist).

I think developers need to correct the program to better support different character sets. And, as a minimum, include in the package file in the correct encoding, since Ubuntu uses UTF-8 by default.

Revision history for this message
Vadim Velikodniy (vadim-velikodniy) wrote :

Oh, sorry, I mixed this mistake with another. But comment that I wrote is true too.

Launching with right locale encoding (LANG=ru_RU.KOI8-R gtypist ru.typ) can solve the trouble with msgid. But the problem with ru.typ remains.

Revision history for this message
Vadim Velikodniy (vadim-velikodniy) wrote :

This error is caused by the following lines (gtypist.c:1902).

  if (strlen(YN) != 3 || YN[1] != '/' || !isupper(YN[0]) || !isupper(YN[2]))
    {
      fprintf( stderr,
        "%s: i18n problem: invalid value for msgid \"Y/N\": %s\n",
        argv0, YN );
      exit( 1 );
    }

This check is wrong for multi-byte encodings (UTF-8, for example).

There is at least one similar fragment (gtypist.c:1911).

I guess strings «Y/N» and «R/N/E» must remain untranslated.

P.S. I use source from Ubuntu 9.04.

Revision history for this message
Stanislav Kogut (stasikos) wrote :

Same with 9.10. Propose genocide of localisators :) Do they test something after localisation?

Revision history for this message
Andrew Grigorev (ei-grad) wrote :

Seems like they tested it only with KOI8-R :-).

Revision history for this message
edam (edam) wrote :

Just to let you know, this has been noticed upstream.

https://savannah.gnu.org/bugs/index.php?33957

Revision history for this message
edam (edam) wrote :

To follow up... The problem is that gtypist doesn't currently support UTF-8. Support for UTF-8 has been added and will be available in the forthcoming 2.9 release.

Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in gtypist (Ubuntu):
status: New → Confirmed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package gtypist - 2.9-3

---------------
gtypist (2.9-3) unstable; urgency=low

  * debian/gtypist.emacsen-startup: Added.
  * debian/gtypist.install: Install gtypist emacs mode.
  * debian/TODO.debian: Added.

gtypist (2.9-2) unstable; urgency=low

  * debian/control (Build-Depends): Removed unused libncurses5-dev.
    (Recommends): Added fortune-cookie-db or typefortune won't be of help.
    (Description): Added a note about fortune-cookie-db.
  * debian/gtypist.doc-base.cs: Added.
  * debian/gtypist.doc-base.en: Ditto.
  * debian/gtypist.doc-base.es: Ditto.
  * debian/gtypist.docs: Added manuals in HTML format.
  * debian/gtypist.install: Ship tools in doc directory (closes: #325565).
  * debian/README.debian: Added notes about fortune cookies and the recently
    added tools.
  * debian/patches/fix_manual_page_issues.patch: Extended. A note in the
    manual page will inform users about the necessity of having fortune
    cookies installed.

gtypist (2.9-1) unstable; urgency=low

  * New upstream release (closes: #620811).
    - Fixes broken UTF-8 support for terminals (closes: #642489;
      LP: #281946, #501541).
    - Fixes minor punctuation error in lesson T14 (closes: #288682).

  * debian/compat: Bumped to dh compat level 7.
  * debian/control: Added Vcs-Browser and Vcs-Svn.
    (Standards-Version): Bumped to 3.9.2.
    (Build-Depends): Added libncursesw5-dev and help2man.
    (Description): Fixed description-synopsis-starts-with-article.
  * debian/copyright: Minor update.
  * debian/gtypist.info: Added.
  * debian/gtypist.lintian-overrides: Updated. Added unusual-interpreter
    for the gtypist interpreter.
  * debian/rules: Rewritten for debhelper 7.
  * debian/README.source: Dropped obsolete file.
  * debian/patches/fix_manual_page_issues.dpatch: Renamed to
    debian/patches/fix_manual_page_issues.patch and updated as quilt patch.
  * debian/patches/00list: Renamed to debian/patches/series and adjusted.
  * debian/source/format: Added for 3.0 format.
 -- Ubuntu Archive Auto-Sync <email address hidden> Mon, 05 Dec 2011 10:59:57 +0000

Changed in gtypist (Ubuntu):
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.