Code review comment for lp:~dpm/ubuntu-calendar-app/rename

Revision history for this message
Olivier Tilloy (osomon) wrote :

So if I understand your code correctly, you’re injecting the "i18n.tr()" calls into a temporary copy of the desktop file. To mimick closer what we’re doing for the core apps using cmake, what about doing the reverse, i.e. having the calls to i18n.tr() already in a desktop.in file, and generating the desktop file from there by removing the calls to i18n.tr()? This would make for a more flexible solution, if in the future we need to translate more strings in the desktop file.

« Back to merge proposal