add appdata file for simon
ClosedPublic

Authored by lueck on Mar 14 2017, 4:39 PM.

Details

Summary

info from documentation

Any app.desktop with Type=Application needs an appdata file?
In this repo that would be simond, afaras, ksimond, sam, scc and sccd

Test Plan

appstreamcli validate simon/org.kde.simon.appdata.xml -> no errors

Diff Detail

Repository
R431 Simon
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
lueck created this revision.Mar 14 2017, 4:39 PM
Restricted Application added a project: Simon. · View Herald TranscriptMar 14 2017, 4:39 PM
Restricted Application added a subscriber: Simon. · View Herald Transcript
mak edited edge metadata.Mar 14 2017, 5:04 PM

Any app.desktop with Type=Application needs an appdata file?
In this repo that would be simond, afaras, ksimond, sam, scc and sccd

Are all of them installed into /usr/share/applications? In general, only the apps you want users to see in software centers need a metainfo file.
You can hide apps in /usr/share/applications from AppStream-data processing tools by adding a X-AppStream-Ignore=true field to the .desktop file.

The file above isn't installed, it should be installed into ${KDE_INSTALL_METAINFODIR} (/usr/share/metainfo)

simon/org.kde.simon.appdata.xml
3

Use "desktop-application" as component-type, to be more modern ;-)

5

Ideally use FSFAP here, unless the project already uses CC0 somewhere, since CC0 requires you to include the license text with the sources, while FSFAP doesn't.
See https://spdx.org/licenses/FSFAP.html for the tiny license.

lueck updated this revision to Diff 12474.Mar 14 2017, 6:21 PM

fix issues spotted by Matthias, many thanks

lueck marked 2 inline comments as done.Mar 14 2017, 6:28 PM
In D5052#95011, @mak wrote:

Any app.desktop with Type=Application needs an appdata file?
In this repo that would be simond, afaras, ksimond, sam, scc and sccd

Are all of them installed into /usr/share/applications? In general, only the apps you want users to see in software centers need a metainfo file.
You can hide apps in /usr/share/applications from AppStream-data processing tools by adding a X-AppStream-Ignore=true field to the .desktop file.

Afaik all desktop files simond, afaras, ksimond, sam, scc and sccd are installed to $KDEDIR/share/applications.
Let Mario decide which apps should users see in software centers

Thanks for review

fux accepted this revision.Mar 20 2017, 12:12 PM

I thought a bit about this and think that only simon itself should be visible in the software centers. The other apps are quite special, mostly packaged together and will most probably not be ported to KF5. Otherwise looks good and please commit.

This revision is now accepted and ready to land.Mar 20 2017, 12:12 PM
mak added a comment.Mar 20 2017, 12:20 PM

Hmm, for some weird reason the diff is the same as before...
In order to have the AppStream metadata generator ignore .desktop files that are in /usr/share/applications/ you must add an X-AppStream-Ignore=true field to them (likely to all except for the org.kde.simon.desktop one).
Otherwise, the other .desktop files might show up in software centers as well.

lueck updated this revision to Diff 12640.Mar 20 2017, 12:39 PM

Fix issues spotted by Matthias (use desktop-application as component-type and FSFAP as project_license)
upload corrected diff, sorry screwed it up :(
add X-AppStream-Ignore=true to simond, afaras, ksimond, sam, scc and sccd desktop files

mak accepted this revision.Mar 20 2017, 12:45 PM

Looks good to me! :-)

fux added a comment.Mar 20 2017, 12:46 PM

Looks good for me too. Go for it.

This revision was automatically updated to reflect the committed changes.