Skip to content

fix(package/linux): update desktop metainfo #3901

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

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

Conversation

joebonrichie
Copy link

Description

  1. By default the .desktop file gets installed to sunshine.desktop, as such provide the launchable tag for it to help appstream-builder "find" the .desktop file due to the id (Sunshine.desktop) not matching.

Rename the id to just @PRODUCT_NAME@ as well since the .desktop suffix is no longer required.

Provide URL as well.

Spec: https://www.freedesktop.org/software/appstream/docs/chap-Metadata.html#tag-launchable

Fixes appstream-builder generation.

  1. Set the metadata_license to FSFAP. To be valid, it has to be one of the licenses vetted by the spec, otherwise appstreamcli-compose will error.

Spec: https://www.freedesktop.org/software/appstream/docs/chap-Metadata.html#tag-metadata_license

Fixes generation with appstreamcli-compose and thus distros which use appstream-generator.

Screenshot

N/A

Issues Fixed or Closed

Follow up to #2006.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Dependency update (updates to dependencies)
  • Documentation update (changes to documentation)
  • Repository update (changes to repository files, e.g. .github/...)

@ReenigneArcher ReenigneArcher changed the title package/linux: Fixup metainfo to keep appstream composers happy fix(package/linux): update desktop metainfo May 24, 2025
Copy link
Member

@ReenigneArcher ReenigneArcher left a comment

Choose a reason for hiding this comment

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

Thank you for the PR! I have a few comments below.

Do you think there is anything else we can commonize between the flatpak version and the standard one?

1. By default the .desktop file gets installed to sunshine.desktop, as
such provide the launchable tag for it to help `appstream-builder`
"find" the .desktop file due to the id (Sunshine.desktop) not matching.

Rename the id to just @PRODUCT_NAME@ as well since the .desktop suffix is
no longer required.

Provide URL as well.

Spec: https://www.freedesktop.org/software/appstream/docs/chap-Metadata.html#tag-launchable

Fixes appstream-builder generation.

2. Set the metadata_license to FSFAP. To be valid, it has to be one of
the licenses vetted by the spec, otherwise `appstreamcli-compose` will
error.

Spec: https://www.freedesktop.org/software/appstream/docs/chap-Metadata.html#tag-metadata_license

Fixes generation with `appstreamcli-compose` and thus distros which use
`appstream-generator`.
@joebonrichie
Copy link
Author

Thank you for the PR! I have a few comments below.

Do you think there is anything else we can commonize between the flatpak version and the standard one?

If possible it's better to consolidate as much as possible the xml metainfo files if they're different. If it passes appstreamcli validate and appstreamcli compose sunshine-install-prefix/. Then it should be fine with flatpak-builder as well as afaiu flatpak-builder uses the same APIs as appstream compose but it's slightly less strict/doesn't enable all checks.

Copy link

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