Typo in skim-scim-pinyin string

Bug #63515 reported by Malcolm Parsons
12
Affects Status Importance Assigned to Milestone
SCIM
Unknown
Unknown
scim-pinyin (Debian)
Fix Released
Unknown
scim-pinyin (Ubuntu)
Fix Released
Low
Unassigned

Bug Description

config/smartpinyinui.cpp:399
"Auto &combin phrase"
s/combin/combine/

Changed in scim-pinyin:
status: Unconfirmed → Rejected
Revision history for this message
Ming Hua (minghua) wrote :

What's happening here? Why the bug submitter rejected the bug without explanation? Is that an accident?

Revision history for this message
Ming Hua (minghua) wrote :

The bug reporter says according to Rosetta FAQ, words starting with & should not be translated, therefore he is not sure if this string is displayed at all to the user. So keep that status rejected until some KDE user confirm that this string is displayed.

Revision history for this message
Malcolm Parsons (malcolm-parsons) wrote :

The string is displayed in KDE.

Changed in scim-pinyin:
status: Rejected → Unconfirmed
Revision history for this message
Sebastian Rode (sebastian-ro-deactivatedaccount) wrote :

Thank you for taking the time to report this bug and helping to make Ubuntu better. You reported this bug a while ago and there hasn't been any activity in it recently. We were wondering is this still an issue for you? Can you try with latest Ubuntu release? Thanks in advance

Changed in scim-pinyin:
status: New → Incomplete
Revision history for this message
Malcolm Parsons (malcolm-parsons) wrote :

scim-pinyin-0.5.91 still has this typo.

Changed in scim-pinyin:
status: Incomplete → New
Revision history for this message
Jean-Baptiste Lallement (jibel) wrote :

 Thanks for reporting this bug and any supporting documentation. Since this bug has enough information provided for a developer to begin work, I'm going to mark it as confirmed and let them handle it from here. Thanks for taking the time to make Ubuntu better!

Changed in scim-pinyin:
status: New → Confirmed
Revision history for this message
Nathan Handler (nhandler) wrote :

scim-pinyin (0.5.91-0ubuntu13) intrepid; urgency=low

  * Add 02smartpinyinui.cpp-typo.dpatch to correct spelling
    of "combine" in skim/setupui/smartpinyinui.cpp (LP: #63515)

 -- Nathan Handler <email address hidden> Wed, 02 Jul 2008 16:44:26 -0500

Revision history for this message
Daniel Holbach (dholbach) wrote :

Arne: can you verify the fix?

Revision history for this message
Arne Goetje (arnegoetje) wrote :

hmm... on my intrepid machine the build seems to be OK... but a 'debuild clean' after the binary build fails. 'dpatch deapply-all' fails to revert 02smartpinyinui.cpp-typo from ./

Could you please review and test locally again?

Thanks

Revision history for this message
Daniel Holbach (dholbach) wrote :

Unsubscribing Ubuntu Sponsors for main from this bug for now.

Changed in scim-pinyin:
importance: Undecided → Low
Revision history for this message
David Stansby (dstansby-deactivatedaccount) wrote :

Here's an updated patch for the new version of scim-pinyin.

Revision history for this message
Martin Pitt (pitti) wrote :

Can you please forward this upstream? If we apply this patch, we break all translations, so it should be (1) applied upstream first, and (2) all existing translations need to be updated for this string change as well.

Changed in scim-pinyin (Debian):
status: Unknown → New
Changed in scim:
importance: Undecided → Unknown
status: New → Unknown
Revision history for this message
Colin Watson (cjwatson) wrote :

This has been forwarded upstream (see the linked bugs), and as Martin noted it would be best to have it fixed there rather than in Ubuntu. I'm taking this off the sponsorship list now. If you have good reason to disagree, it's easy to resubscribe the sponsors team.

tags: added: patch-forwarded-upstream
removed: patch
Changed in scim-pinyin (Debian):
status: New → Fix Committed
tags: added: patch-forwarded-debain
Robert Roth (evfool)
tags: added: patch-forwarded-debian
removed: patch-forwarded-debain
Changed in scim-pinyin (Debian):
status: Fix Committed → New
Revision history for this message
Daniel Polehn (dpolehn-gmail) wrote :

Looks like this was fixed upstream.

Changed in scim-pinyin (Ubuntu):
status: Confirmed → Fix Committed
Revision history for this message
David Stansby (dstansby-deactivatedaccount) wrote :
Changed in scim-pinyin (Ubuntu):
status: Fix Committed → Confirmed
Changed in scim-pinyin (Debian):
status: New → Fix Released
Revision history for this message
Rolf Leggewie (r0lf) wrote :

the skim source directory was dropped in 0.5.92

Changed in scim-pinyin (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.