[Kde-pim] Review Request 113252: Extract messages from javascript wizards

Luigi Toscano luigi.toscano at tiscali.it
Wed Nov 13 23:40:24 GMT 2013



> On Nov. 13, 2013, 11:38 p.m., Commit Hook wrote:
> > This review has been submitted with commit 44eb1e341cfcb4711100b8b0510a5855579cda57 by Luigi Toscano to branch master.

I pushed by mistake on master: is it a problem if I cherry-pick on 4.12? 


- Luigi


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/113252/#review43640
-----------------------------------------------------------


On Nov. 13, 2013, 11:38 p.m., Luigi Toscano wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/113252/
> -----------------------------------------------------------
> 
> (Updated Nov. 13, 2013, 11:38 p.m.)
> 
> 
> Review request for KDEPIM.
> 
> 
> Repository: kdepim-runtime
> 
> 
> Description
> -------
> 
> Translate the javascript files used in the wizards that were not translated before.
> I was going to commit this directly (as it does not impact the code, and it fixes a long-standing issue), but this change depends on a kdelibs patch which has been committed today (https://git.reviewboard.kde.org/r/113218/), which means kdelibs 4.11.3. The code in this review adds some calls to qsTr(), and when that kdelibs patch is not applied the wizard scripts exit with a run time error (and a blank window). So the code in this review adds an implicit dependency on kdelibs 4.11.3; the cmake code says "kdelibs 4.10, kdepimlibs 4.11". Would it be acceptable?
> 
> 
> Diffs
> -----
> 
>   accountwizard/wizards/tine20/Messages.sh 30a1fec 
>   accountwizard/wizards/tine20/tine20wizard.es 6970b1d 
>   resources/imap/wizard/Messages.sh 8fbc16c 
>   resources/imap/wizard/imapwizard.es dcab06f 
>   resources/kolabproxy/wizard/Messages.sh 5654648 
>   resources/kolabproxy/wizard/kolabwizard.es 842e757 
>   resources/maildir/wizard/Messages.sh 69bb8e9 
>   resources/maildir/wizard/maildirwizard.es af61f8a 
>   resources/mbox/wizard/Messages.sh d720fde 
>   resources/mbox/wizard/mailboxwizard.es 286d4c0 
>   resources/pop3/wizard/Messages.sh fee8e77 
>   resources/pop3/wizard/pop3wizard.js 47262e4 
> 
> Diff: http://git.reviewboard.kde.org/r/113252/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Luigi Toscano
> 
>

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/



More information about the kde-pim mailing list