GNOME Bugzilla – Bug 708448
consider adding appdata
Last modified: 2014-02-25 16:14:34 UTC
http://people.freedesktop.org/~hughsient/appdata/ Not sure it's really necessary, since most likely g-t will be installed already, by default.
It's recommendable to have it, as this application is shown in GNOME software. If you can provide the texto for the description, I can send you a patch. Thanks!
Found this example in Hughsie's github (URL in wiki was wrong): https://github.com/hughsie/fedora-appstream/blob/master/appdata-extra/desktop/gnome-termainal.appdata.xml It's ok to use it, adding email in the <updatecontact> field?
Would be great to get this in for 3.12
Christian, it's ok to include you in the updatecontact field? If so, I can commit this change before string freeze
I would suggest to just go without the updatecontact
Created attachment 270100 [details] [review] Add appdata file Here is the patch to add the AppData file. Please review it and, if it's ok, I'll ask for a freeze break to commit it. Thanks!
I added a skeleton appdata file in git now. +A staple for power users, the terminal helps you get in touch with your hacker roots, +and makes it easy to get under the hood and modify your system. No way I'm adding this :-)
BTW: I had wanted to use ITS to translate the appdata file, but ITS isn't integrated yet with the gnome l10n workflow, so I fell back to using intltool now for.
So, can I modify this skeleton with the text suggested by Hugsie? (I'd prefer tou use yours, but maybe is not a good idea xDD ) Thanks!
It's only a skeleton, intended to be changed again, so yes, do post a patch for the <description>. However the text used in the URL in comment 2 is inacceptable. The text needs to be absolutely factual, not read like an ad.
Created attachment 270266 [details] [review] AppData File Really don't like the description, but didn't know what to write... please feel free to modify it if you want
s/to made your life easier//. With that change, ok to commit after string freeze approval.
Created attachment 270284 [details] [review] Patch updated Ok, here is the updated patch. I'm asking for a freeze break. Thanks!
Review of attachment 270284 [details] [review]: Patch commited.