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

Metainfo files are ignored for Elementary OS apps #88

Closed
City-busz opened this issue Apr 16, 2021 · 2 comments · Fixed by #91
Closed

Metainfo files are ignored for Elementary OS apps #88

City-busz opened this issue Apr 16, 2021 · 2 comments · Fixed by #91

Comments

@City-busz
Copy link
Contributor

I usually get 'no-metainfo' warnings for Elementary OS applications. The metainfo file is ignored by appstream-generator, and just the desktop file is processed. Tested apps (on Arch Linux) with their metainfo files:

It seems that when I specify the corresponding desktop file explicitly in the metainfo file with the <launchable type="desktop-id"> tag, it solves the problem.

Elementary OS applications usually use the same name for desktop and metainfo files , e.g.:

  • desktop file: /usr/share/applications/com.github.davidmhewitt.torrential.desktop
  • metainfo file: /usr/share/metainfo/com.github.davidmhewitt.torrential.appdata.xml

So could appstream-generator handle these applications better, or is the <launchable> tag necessarily for the pairing?

@ximion
Copy link
Owner

ximion commented Apr 19, 2021

The <launchable> tag is mandatory, but we did have some workaround in place to just check "ID + .desktop" as heuristic for finding desktop-entry files. That probably broke in the move to more ascompose components, I'll have a look if it's feasible to add this feature back.
Adding launchable tags is a good idea in any case though :-)

@City-busz
Copy link
Contributor Author

I found the problem! cid ~ ".desktop" is not removed from desktop list, so it processed twice, and the latter overwrites the previous data loaded from metainfo. See #91 for a simple fix.

ximion pushed a commit that referenced this issue Jun 22, 2021
)

Otherwise it processed twice, and overrides data from metainfo file.

Fixes #88
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 a pull request may close this issue.

2 participants