KEMBAR78
I18N: Add script to automatically generate scummvm.appdata.xml from .po files by ankushdutt · Pull Request #4826 · scummvm/scummvm · GitHub
Skip to content

Conversation

@ankushdutt
Copy link
Member

This PR adds a script to support multilanguage in scummvm.appdata.xml. The multilanguage data is extracted from po/*.po files.

Related Discord discussion: https://discord.com/channels/581224060529148060/1086942326489895033/1086976457311531058

Copy link
Member

@sev- sev- left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, but couple of changes are required.

description_elem.appendChild(p_elem)

# paragraph 3
p_elem = doc.createElement("p")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please, turn it into a simpler template. Working with current soup of in-code tags is not practical

Copy link
Member Author

@ankushdutt ankushdutt Mar 21, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed and changed the code to have an in-source copy of the template and added code to substitute specific keywords with relevant translations.

<project_license>GPL-3.0-or-later</project_license>
<name>ScummVM</name>
<summary>Interpreter for numerous adventure games and role-playing games</summary>
<summary xml:lang="ru">Интерпретатор множества приключенческих и ролевых игр</summary>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These, and other language tags should be sorted alphabetically, so their sequence stays consistent

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed

@ankushdutt ankushdutt marked this pull request as draft March 21, 2023 10:03
@ankushdutt ankushdutt changed the title I18N: Add script to automatically generate scummvm.appdata.xml from .po files [WIP] I18N: Add script to automatically generate scummvm.appdata.xml from .po files Mar 21, 2023
@ankushdutt ankushdutt marked this pull request as ready for review March 21, 2023 11:31
@ankushdutt ankushdutt changed the title [WIP] I18N: Add script to automatically generate scummvm.appdata.xml from .po files I18N: Add script to automatically generate scummvm.appdata.xml from .po files Mar 21, 2023
@ankushdutt ankushdutt requested review from sev- and removed request for sev- March 21, 2023 11:32
@sev-
Copy link
Member

sev- commented Mar 21, 2023

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants