<table><tr><td style="">mak requested changes to this revision.<br />mak added inline comments.<br />This revision now requires changes to proceed.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D13772">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D13772#inline-72406">View Inline</a><span style="color: #4b4d51; font-weight: bold;">org.kde.ksysguard.appdata.xml:2</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);"><span style="color: #304a96"><?xml version="1.0" encoding="utf-8"?></span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);"><span style="color: #00702a"><component</span> <span style="color: #354bb3">type=</span><span style="color: #766510">"desktop"</span><span style="color: #00702a">></span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">  <span style="color: #00702a"><name></span>KSysGuard<span style="color: #00702a"></name></span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Nitpick: Use <tt style="background: #ebebeb; font-size: 13px;">desktop-application</tt></p></div></div><br /><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D13772#inline-72407">View Inline</a><span style="color: #4b4d51; font-weight: bold;">org.kde.ksysguard.appdata.xml:7</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">  <span style="color: #00702a"><id></span>org.kde.ksysguard<span style="color: #00702a"></id></span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">  <span style="color: #00702a"><metadata_license></span>CC0-1.0<span style="color: #00702a"></metadata_license></span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">  <span style="color: #00702a"><project_license></span>GPL-2.0+<span style="color: #00702a"></project_license></span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Maybe using <tt style="background: #ebebeb; font-size: 13px;">FSFAP</tt> as license is better here, so the sources don't have to ship a copy of the CC0 license.</p></div></div><br /><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D13772#inline-72408">View Inline</a><span style="color: #4b4d51; font-weight: bold;">org.kde.ksysguard.appdata.xml:18</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">    <span style="color: #00702a"><screenshot</span> <span style="color: #354bb3">type=</span><span style="color: #766510">"default"</span><span style="color: #00702a">></span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">      <span style="color: #00702a"><image</span> <span style="color: #354bb3">type=</span><span style="color: #766510">"source"</span><span style="color: #00702a">></span>https://www.kde.org/images/screenshots/ksysguard.png<span style="color: #00702a"></image></span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">      <span style="color: #00702a"><image</span> <span style="color: #354bb3">type=</span><span style="color: #766510">"thumbnail"</span><span style="color: #00702a">></span>https://www.kde.org/images/screenshots/resized/ksysguard.png<span style="color: #00702a"></image></span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Thumbnails are generally thrown away be the metadata extractor on the Linux distro side, so you can really drop the "type" property of the image tag here and just add the source image.</p></div></div><br /><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D13772#inline-72409">View Inline</a><span style="color: #4b4d51; font-weight: bold;">org.kde.ksysguard.appdata.xml:28</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">  <span style="color: #00702a"></mimetypes></span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">The component is missing a <tt style="background: #ebebeb; font-size: 13px;">launchable</tt> tag: <a href="https://www.freedesktop.org/software/appstream/docs/chap-Metadata.html#tag-launchable" class="remarkup-link" target="_blank" rel="noreferrer">https://www.freedesktop.org/software/appstream/docs/chap-Metadata.html#tag-launchable</a><br />
The categories and mimetypes lists can be dropped, as those will be automatically extracted from the desktop-file mentioned in the launchable tag (if they are not dropped, the metainfo file will *override* the desktop file contents, which is likely not the desired behavior here)</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R106 KSysguard</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D13772">https://phabricator.kde.org/D13772</a></div></div><br /><div><strong>To: </strong>ngraham, Plasma, apol, mak<br /><strong>Cc: </strong>mak, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>