String needs a translation comment and use of %(var)s python variables

[David Planella]

Binary package hint: gdebi

This particular string

https://translations.edge.launchpad.net/ubuntu/jaunty/+source/gdebi/+pots/gdebi/ca/78/+translate

Would need:

a) A comment for translators to indicate what each variable corresponds to

b) The usage of %(var)s python string variables, since otherwise there is no possibility to reorder the variables in the translation, which some languages will need to do.

[Michael Vogt (mvo)]

Thanks for your bugreport.

Is this still a issue? The current code reads:

# TRANSLATORS: the first '%s' is the package that breaks, the second the dependency that makes it break, the third the releation (e.g. >=) and the latest the version for the releation
self._failureString += _("Breaks existing package '%s' dependency %s (%s %s)\n") % (pkg.name, dep.name, dep.relation, dep.version)

[David Planella (dpm)]

Hi Michael,

The comment improved the situation, but some languages need to rearrange the order of the strings to make the translation readable, which currently is not possible using %s strings, only %(variable)s variables allow this.

Here's a branch with an attempt to fix this. I haven't tested it yet, since I was not sure how to make gdebi show any of the particular strings affected.

[David Planella (dpm)]

I'm resubmitting the merge proposal, as in the fix I missed deleting one of the original lines containing the string

[Launchpad Janitor (janitor)]

This bug was fixed in the package gdebi - 0.6.0

---------------
gdebi (0.6.0) unstable; urgency=low

  * data/gdebi.desktop.in, data/gdebi-kde.desktop.in:
    - Remove Encoding field, deprecated.
  * debian/control:
    - Set XS-Python-Version to all (Closes: #570560).
    - Differentiate short descrriptions for gdebi and gdebi-kde.
  * GDebi/DebPackage.py:
    - improve i18n by using %(var)s LP: #403632, thanks to
      David Planella
  * data/gdebi.ui, GDebi/DebPackage.py:
    - alignment fixes in the status label
  * GDebi/SimpleGtkbuilderApp.py:
    - set translation domain here explicitly instead of in gdebi.ui
      (the later is often cleared by glade for some reason)
  * .bzr-builddeb/default.conf:
    - build as native package again
  * unified release for Debian&Ubuntu

gdebi (0.5.9debian3) unstable; urgency=low

  * gdebi, GDebi/GDebiCli.py:
    - Correctly parse translated "y" characters (Closes: #497821).
  * GDebi/GDebiCli.py:
    - Allow install of Debian binary packages with uncommon extensions
      or missing one (Closes: #472693).
  * po/es.po:
    - Update Spanish translation, by Omar Campagne (Closes: #556927).
  * README:
    - No longer mention packages are Ubuntu-specific, they are available
      in Debian too (Closes: #453709).
  * debian/control:
    - Do not build-depend on python-dev, python is enough.
 -- Michael Vogt <email address hidden> Wed, 10 Mar 2010 12:04:46 +0000