Keywords from .desktop file not merged into catalogue data if metainfo also contains keywords

[Alexander-Wilms]

Keywords from .desktop file not merged into catalogue data if metainfo also contains keywords

[bbhtt]

@bbhtt might know which version of appstream is currently used for Flathub.

1.0.3.

I do have a faint memory of keywords being merged but maybe that's with appstream-glib.

[ximion]

I am pretty sure the intent is that the contents of the metainfo file override stuff from the desktop file, and that this was actually changed nearly a decade ago due to issues with the merging. So, I am pretty sure this works as intended.

Why would you want to define keywords in two separate places?

[Alexander-Wilms]

I would prefer to only define them in the .desktop file so they get used for the search in application launchers as well as software centers. See also #636

But since metainfo files also support keywords, since not every component has a .desktop file, I'd expect them to get merged.

Right now, some apps use the keywords from the .desktop file and some do not, which I think is confusing.

[bbhtt]

If they are overridden, the spec should mention it. Seems like I was relying on pre-libappstream behaviour here. I fixed the docs for now.