String needs a translation comment and use of %(var)s python variables
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
| Ubuntu Translations |
Low
|
Unassigned | ||
| gdebi (Ubuntu) |
Undecided
|
Unassigned |
Bug Description
Binary package hint: gdebi
This particular string
https:/
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.
Related branches
- Michael Vogt: Approve on 2010-02-25
- David Planella (community): Resubmit on 2010-02-11
- Ubuntu branches: Pending requested 2010-02-11
-
Diff: 23 lines (+3/-3)1 file modifiedGDebi/DebPackage.py (+3/-3)
tags: | added: i18n |
David Planella (dpm) wrote : | #2 |
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.
Changed in gdebi (Ubuntu): | |
status: | Incomplete → Confirmed |
Changed in ubuntu-translations: | |
status: | New → In Progress |
importance: | Undecided → Low |
assignee: | nobody → David Planella (dpm) |
Changed in gdebi (Ubuntu): | |
status: | Confirmed → Fix Committed |
Changed in ubuntu-translations: | |
status: | In Progress → Fix Committed |
assignee: | David Planella (dpm) → nobody |
David Planella (dpm) wrote : | #3 |
I'm resubmitting the merge proposal, as in the fix I missed deleting one of the original lines containing the string
Changed in ubuntu-translations: | |
status: | Fix Committed → In Progress |
assignee: | nobody → David Planella (dpm) |
Launchpad Janitor (janitor) wrote : | #4 |
This bug was fixed in the package gdebi - 0.6.0
---------------
gdebi (0.6.0) unstable; urgency=low
* data/gdebi.
- Remove Encoding field, deprecated.
* debian/control:
- Set XS-Python-Version to all (Closes: #570560).
- Differentiate short descrriptions for gdebi and gdebi-kde.
* GDebi/DebPackag
- improve i18n by using %(var)s LP: #403632, thanks to
David Planella
* data/gdebi.ui, GDebi/DebPackag
- alignment fixes in the status label
* GDebi/SimpleGtk
- set translation domain here explicitly instead of in gdebi.ui
(the later is often cleared by glade for some reason)
* .bzr-builddeb/
- 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
Changed in gdebi (Ubuntu): | |
status: | Fix Committed → Fix Released |
Changed in ubuntu-translations: | |
status: | In Progress → Fix Released |
assignee: | David Planella (dpm) → nobody |
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)