[kde-edu]: i18n issues with step

[Albert Astals Cid]

A Divendres 05 Desembre 2008, Burkhard Lück va escriure:
> Am Freitag 05 Dezember 2008 05:04:30 schrieb Henry de Valence:
> > On Thu December 4 2008 6:14:07 pm Albert Astals Cid wrote:
> > > A Dimecres 03 Desembre 2008, Burkhard Lück va escriure:
> > > > Am Dienstag 02 Dezember 2008 23:32:16 schrieb Albert Astals Cid:
> > > > > Well, actually it's not what KDE does but what translators can do
> > > > > (Transcript), because transcript can not work on tr() so you should
> > > > > know when translating a .po what you can and you can't do.
> > > >
> > > > Yes, that indeed makes sense.
> > > >
> > > > So attached a fix to split the catalogs for step, one called
> > > > "step.pot" with the kde i18n messages and another called
> > > > "step_qt.pot" with all qt messages.
> > > >
> > > > Please check.
> > > >
> > > > Before committing the translators have to be warned and need an
> > > > explanation about the messages catalogs "foo_qt".
> > > >
> > > > Oh, and step also needs a patch to include the new catalog.
> > >
> > > Done, fixed the stepcore Messages.sh to use the correct XGETTEXT_QT
> > > definition.
> > >
> > > > Marble and kalzium in kdeedu have the same problem as step, all
> > > > messages either i18n() or tr() are extracted into one catalog.
> > > >
> > > > Their catalogs should be splitted too.
> > >
> > > Can you work on such patches?
>
> Attached one patch for marble and one for kalzium. Please commit only in
> coordination with Albert.
>
> marble patch:
> Works for the kde version, but the qtonly version has to be fixed to use
> the second catalog "marble_qt", but I don't know how to do this.
> Tosten, Inge, or Henry, please fix this issue, when the message split is
> commited.
>
> kalzium patch:
> I have no idea why kalzium uses tr() calls, is there a qt version of
> kalzium? What's the reason using tr() calls?
>
> If there is no good reason for using tr() calls, a better patch would be to
> change the tr() calls to i18n() calls and extract the messages into one
> catalog then.
>
> Carsten?
>
> > I could be wrong, but AFAIR Marble uses only tr() and in a KDE build it
> > converts them into i18n(), instead of having both.
>
> You are wrong, see attached patch.

Commited both, fixing kalzium patch where you had forgot to add the 
XGETTEXT_QT line

Albert