Appstream data for several programs missing languages about flathub

[ximion]

Yea seems to be the symlinks

I just checked the code, symlinks are indeed ignored, apparently for performance reasons... I doubt it would hurt performance that much if we quickly checked if the symlink target exists... I'll have a look at this later :-)

[bbhtt]

Seems like something is going wrong in appstream but I'm not sure. Building meld and adding some debugs, this part https://github.com/ximion/appstream/blob/e92858da0310be987950fa5c22a12fc0a747d17b/compose/asc-utils-l10n.c#L459-L463 is outputting only two percentage datas somehow en_ca and en_gb.

Maybe it has something to do with symlinks because en_ca and en_gb are not symlinks in build/files/share/locale, every other language is a symlink to build/files/share/runtime/locale/$lang_code setup by flatpak-builder. EN and C are excluded from "locale migration"

The appstream metadata is itself getting translated though.

Hey @ximion any ideas ?

[bbhtt]

Yea seems to be the symlinks, if I disable "locale migration" by doing:

diff --git a/org.gnome.meld.json b/org.gnome.meld.json
index ad962d1..545be37 100644
--- a/org.gnome.meld.json
+++ b/org.gnome.meld.json
@@ -3,6 +3,7 @@
     "runtime": "org.gnome.Platform",
     "runtime-version": "45",
     "sdk": "org.gnome.Sdk",
+    "separate-locales": false,
     "command": "meld",
     "cleanup": [
         "/include",

Then the catalogue data has percentage data for all languages.

[ximion]

Very odd... My initial impulse would be to look at the original MetaInfo data and double-check that the translation tag is set correctly for the respective file.

If the symlink targets exist, I see no reason why the locale data wouldn't be read.

[ximion]

Sorry for the long wait, I have a lot of stuff on my plate right now. This patch should fix this issue, and there will likely be a new AppStream release this month as well.

[bbhtt]

Thank you. There's no hurry, as apps will need to get rebuilt to refresh the catalogue data, anyways. I'll wait for the release.