Skip to content

common/Scripts: Add a script to validate AppStream #5726

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

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

EbonJaeger
Copy link
Member

@EbonJaeger EbonJaeger commented Jun 2, 2025

Summary

Adds a script to validate an AppStream metainfo XML file. It uses a bunch of overrides sourced from Flathub. I decided to use these to try to maintain the best compatibility; the idea is if a metainfo file works for Flathub, it should work for us.

Some exceptions are commented out as they rely on patches to appstream (I think?), and I haven't looked into locating those yet. We can decide to pursue that at a later time and uncomment them, or just drop them.

Signed-off-by: Evan Maddock [email protected]

Test Plan

Run the script against some metainfo XML files in my system, and in the package repository files.

Checklist

  • Package was built and tested against unstable
  • This change could gainfully be listed in the weekly sync notes once merged

@github-project-automation github-project-automation bot moved this to Triage in Solus Jun 2, 2025
@EbonJaeger EbonJaeger force-pushed the add-appstream-validator branch from d359261 to fe808b7 Compare June 2, 2025 21:18
Adds a script to validate an AppStream metainfo XML file. It uses a
bunch of overrides sourced from Flathub. I decided to use these to try
to maintain the best compatibility; the idea is if a metainfo file works
for Flathub, it should work for us.

Some exceptions are commented out as they rely on patches to appstream
(I think?), and I haven't looked into locating those yet. We can decide
to pursue that at a later time and uncomment them, or just drop them.

Signed-off-by: Evan Maddock <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Triage
Development

Successfully merging this pull request may close these issues.

1 participant