-
-
Notifications
You must be signed in to change notification settings - Fork 8
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 an appdata file #8
Conversation
Nice, thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The file should go into the linux
subfolder
panini.appdata.xml
Outdated
</description> | ||
<!-- no screenshots --> | ||
<url type="homepage">https://lazarus-pkgs.github.io/lazarus-pkgs/panini.html</url> | ||
<updatecontact>tksharpless@gmail.com</updatecontact> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thomas isn't the main contact for this anymore, please add my email instead (you can see it in git log
).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done, thanks.
Yeh, ideally, the project file should put it in This is the Fedora documentation on Appdata, for reference: https://fedoraproject.org/wiki/Packaging:AppData |
</description> | ||
<!-- no screenshots --> | ||
<url type="homepage">https://lazarus-pkgs.github.io/lazarus-pkgs/panini.html</url> | ||
<updatecontact>jubalh AT iodoru DOT org</updatecontact> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is it okay to write the email in such a format? I dont much about appdata, so not sure which formats the progams that parse that file will read it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think the e-mail address is parsed at all currently. I did validated it and didn't get an error:
appstream-util validate-relax linux/panini.appdata.xml
linux/panini.appdata.xml: OK
So I reckon it's OK.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok
Please squash the first 3 commits into one. |
3422a81
to
f7b38aa
Compare
Thanks for this! |
This was reported as https://sourceforge.net/p/pvqt/feature-requests/5/