Skip to content

Add AppData manifest #1445

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

Merged
merged 4 commits into from
Dec 19, 2019
Merged

Add AppData manifest #1445

merged 4 commits into from
Dec 19, 2019

Conversation

tim77
Copy link
Contributor

@tim77 tim77 commented Dec 17, 2019

Add initial desktop file and AppData manifest file for modern package managers like GNOME Software/Discover KDE. It will be easier to discover and install Chatterino for users in future. Also there is cool donating button. :)

https://www.freedesktop.org/software/appstream/docs/chap-Metadata.html

If you think name scheme is weird please check this
https://www.freedesktop.org/software/appstream/docs/chap-Metadata.html#sect-Metadata-GenericComponent

This appdata and UUID name is also mandatory if you plan distibute your app in future as Flatpak package.

Rename desktop file to proper UUID name scheme
@@ -0,0 +1,35 @@
<?xml version="1.0" encoding="UTF-8"?>
<!-- Copyright 2019 Artem Polishchuk <ego.cordatus@gmail.com> -->
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is this compatible with the MIT license (used by Chatterino 2)?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yep, this appdata under CC0-1.0 license which have no restrictions at all. Just a credit to author. I followed this guidelines
https://docs.fedoraproject.org/en-US/packaging-guidelines/AppData/#_appdata_xml_file_creation

Copy link
Member

@pajlada pajlada 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 your contribution, just these two small points and I'm ready to accept the PR.

@@ -0,0 +1,35 @@
<?xml version="1.0" encoding="UTF-8"?>
<!-- Copyright 2019 Artem Polishchuk <ego.cordatus@gmail.com> -->
Copy link
Member

Choose a reason for hiding this comment

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

Curious about best practices in terms of copyrighting contributions like this - generally any contribution would fall under the project license (MIT).
I'd prefer we just skip the copyright notice for now. I understand this makes our file not validate the strictest check, but that doesn't seem to be a real concern.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No prob. I'll fix this shortly.

chatterino.pro Outdated
@@ -509,7 +509,7 @@ linux:isEmpty(PREFIX) {
}

linux {
desktop.files = resources/chatterino.desktop
desktop.files = resources/com.chatterino.chatterino2.desktop
Copy link
Member

Choose a reason for hiding this comment

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

The "application id" we use for other OSes has been com.chatterino.chatterino - I think we should stay consistent. Please update this in the other changes you've made too.

Copy link
Member

@pajlada pajlada left a comment

Choose a reason for hiding this comment

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

Neat, thanks for the contribution!

@pajlada pajlada merged commit 142e967 into Chatterino:master Dec 19, 2019
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