Skip to content

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

Draft
EbonJaeger wants to merge 1 commit intomainfrom
add-appstream-validator
Draft

common/Scripts: Add a script to validate AppStream#5726
EbonJaeger wants to merge 1 commit intomainfrom
add-appstream-validator

Conversation

@EbonJaeger
Copy link
Copy Markdown
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 maddock.evan@vivaldi.net

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 <maddock.evan@vivaldi.net>
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