clbr / radeontop

GNU General Public License v3.0
804 stars 70 forks source link

appdata for radeontop #50

Closed luyatshimbalanga closed 7 years ago

luyatshimbalanga commented 7 years ago

Following the appstream guideline needed for Software Center found on major distribution like Ubuntu and Fedora, here is radeontop.metainfo.xml.txt. Remove the txt extension and verify if the information are correct.

clbr commented 7 years ago

The license is GPL3, not +. There is some weird utf-8 character at the start of the id line. Finally, I do not understand what it means for it to extend Gimp. Please clarify these, otherwise I can't add a file I don't use nor understand.

luyatshimbalanga commented 7 years ago

radeontop.metainfo.xml.txt On 06/08/17 12:48 AM, clbr wrote:

The license is GPL3, not +. There is some weird utf-8 character at the start of the id line. Finally, I do not understand what it means for it to extend Gimp. Please clarify these, otherwise I can't add a file I don't use nor understand.

The license line is fixed. The id line is from appstream guideline which needs to be unique. You can change it. The extends line is removed as it was intended for addon and the component is now console-application.

To validate the metainfo, install libappstream-glib (the name may differ from distribution) and enter the command

appstream-util validate-relax --nonet path/radeontop.metainfo.xml

Further info for appstream is on https://freedesktop.org/software/appstream/docs/chap-Metadata.html . Hope it helps.

clbr commented 7 years ago

Added.