D14411: Download and use localization files for KStars Lite
Pino Toscano
noreply at phabricator.kde.org
Thu Jul 26 21:04:47 BST 2018
pino requested changes to this revision.
pino added a comment.
This revision now requires changes to proceed.
This does not look good for different reasons:
- the KI18n framework is marked as working on Android (and looking at its git history, there were fixes for it); hence, if it still does not work, please fix it instead (so it will work for kstars //any// any other KI18n-based application)
- if you really need to define i18n() as gettext(), then wrap it as QString::fromUtf8(), otherwise the encoding is completely wrong (implicit char* -> QString cast uses latin1/ascii)
- it mixes unrelated changes (like header reshuffling) -- please make sure each commit is **not** an aggregate of many unrelated changes!
- all the various string changes, like the introduction of string puzzles, are big no-no
Please get KI18n to work: all these changes are intrusive, and wrong, workarounds to that.
REPOSITORY
R321 KStars
REVISION DETAIL
https://phabricator.kde.org/D14411
To: ckertesz, mutlaqja, pino
Cc: pino, kde-edu, narvaez, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20180726/67de3fc0/attachment.html>
More information about the kde-edu
mailing list