Skip to content

common: Add package check for metainfo file name#9151

Open
EbonJaeger wants to merge 1 commit into
mainfrom
add-lint-for-metainfo-file
Open

common: Add package check for metainfo file name#9151
EbonJaeger wants to merge 1 commit into
mainfrom
add-lint-for-metainfo-file

Conversation

@EbonJaeger

Copy link
Copy Markdown
Member

Summary
We want to be able to catch when a package contains AppStream metainfo files that don't follow the correct naming convention. If they don't, they may not be picked up during the generation of the AppStream catalog.

Signed-off-by: Evan Maddock maddock.evan@vivaldi.net

Test Plan

  1. Modify by hand the package.yml and pspec_x86_64 files for gnome-software and discover to bump the package release.
  2. Run go-task check and see no warnings.
  3. Modify the name of a metainfo file in one of the pspec files to be an invalid name.
  4. Run go-task check and see a warning about the metainfo file name.

Checklist

  • Package was built and tested against unstable
  • This change could gainfully be listed in the weekly sync notes once merged
  • I agree to license this contribution and all my previous contributions under the licensing terms in LICENSE.md and have the power and authority to grant those licenses.

**Summary**
We want to be able to catch when a package contains
AppStream metainfo files that don't follow the
correct naming convention. If they don't, they may
not be picked up during the generation of the
AppStream catalog.

Signed-off-by: Evan Maddock <maddock.evan@vivaldi.net>
@EbonJaeger EbonJaeger force-pushed the add-lint-for-metainfo-file branch from 7a4a1b6 to cd53d43 Compare June 5, 2026 23:47
@Jaredy899

Copy link
Copy Markdown
Contributor

I think we need to add something about appstreamcli validate because I've forgotten to do this. Is this something we can add?

@EbonJaeger

Copy link
Copy Markdown
Member Author

Where? That sounds outside the scope of this PR.

@clintre

clintre commented Jun 7, 2026

Copy link
Copy Markdown
Member

@Jaredy899 Part of the problem is that this issue this aims to resolve was not caught by appstreamcli validate

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.

3 participants