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

Add AppStream #2649

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

Add AppStream #2649

wants to merge 1 commit into from

Conversation

JakobDev
Copy link
Contributor

@JakobDev JakobDev commented Sep 1, 2022

This PR adds a AppStream file for Linux. AppStream contains the data for Software centers on Linux (e.g.g Gnome Software or KDE Discover) like Description, Screenshots, License, Links etc. If a App provides no AppStream file, the data from the .desktop files are used, which is missing a lot of things of course.

This AppStream is currently missing some screenshots, as I can't find some good. Please provide the Link to at least one Screenshot, so I can add them before merging.

@endrift
Copy link
Member

endrift commented Sep 9, 2022

Is this meant to be distributed along with the application? There's no way I'm going to remember to update the release list every single time I make a new release...there are multiple versions of mGBA that just straight up show the wrong release number in places since I forgot to update it for a minor release.

@JakobDev
Copy link
Contributor Author

JakobDev commented Sep 9, 2022

Is this meant to be distributed along with the application?

Yes

There's no way I'm going to remember to update the release list every single time I make a new release.

Maybe this can be automated with GitHub actions are this can be set during the build with CMake.

@Jan200101
Copy link
Contributor

Hey, is there any plan to continue this PR?
I've implemented some cmake logic that automatically populates the releases list through the git cli, though it doesn't include a type or the tag url
Jan200101@2cedad1

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.

3 participants