Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

appdata: translate=no properties #133

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

yakushabb
Copy link
Contributor

It appears that the appstream project no longer supports translatable=no properties, and gettext extract the translatable=no marked strings as translatable.

I opened an issue to inform about the situation, but translatable=no properties are not accepted by developers. You can find the issue here: ximion/appstream#623

Please test your script or string extraction process before merging this PR.

In MetaInfo files, each individual paragraph of a description
(or enumerated entry) is translated individually, however,
you can only exclude the complete block from being translated
by adding translate="no" to the description element.

Source: https://freedesktop.org/software/appstream/docs/sect-Quickstart-Translation.html

It appears that the appstream project no longer supports
`translatable=no` properties, and gettext extract the
`translatable=no` marked strings as translatable.

I opened an issue to inform about the situation, but `translatable=no`
properties are not accepted by developers. You can find the issue
here: ximion/appstream#623

Please test your script or string extraction process
before merging this PR.

> In MetaInfo files, each individual paragraph of a description
> (or enumerated entry) is translated individually, however,
> you can only exclude the complete block from being translated
> by adding `translate="no"` to the description element.

Source: https://freedesktop.org/software/appstream/docs/sect-Quickstart-Translation.html
@FineFindus
Copy link
Owner

FineFindus commented May 5, 2024

Sorry for the long pause.

Please test your script or string extraction process before merging this PR.

I'm using the scripts/make-pot.sh script, after running it with the changes, it seems like it does not recognize the new property.

@yakushabb
Copy link
Contributor Author

Your metainfo rules might not be updated due to gettext not yet accepted the patch.

You can find your metainfo related files, back up them, then update from the following appstream files.

https://github.com/ximion/appstream/blob/main/data/its/metainfo.loc
https://github.com/ximion/appstream/blob/main/data/its/metainfo.its

Related topic/comments:

@FineFindus
Copy link
Owner

I'm a bit confused about what files I should replace?
Is there any ETA on when the patch may be included? I feel like that would be a better solution.

@yakushabb
Copy link
Contributor Author

No ETA. Also no action from gettext side.

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