<table><tr><td style="">mak added a comment.
</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/D6628">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
color: #6B748C;
font-style: italic;
margin: 4px 0 12px 0;
padding: 8px 12px;
background-color: #F8F9FC;">
<div style="font-style: normal;
padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D6628#676720" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: line-through;">D6628#676720</a>, <a href="https://phabricator.kde.org/p/sitter/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@sitter</a> wrote:</div>
<div style="margin: 0;
padding: 0;
border: 0;
color: rgb(107, 116, 140);"><p>These aren't SPDX identifiers but kaboutlicense keywords <a href="https://api.kde.org/frameworks/kcoreaddons/html/kaboutdata_8cpp_source.html#l00397" class="remarkup-link" target="_blank" rel="noreferrer">https://api.kde.org/frameworks/kcoreaddons/html/kaboutdata_8cpp_source.html#l00397</a></p></div>
</blockquote>
<p>Makes sense.<br />
<a href="https://phabricator.kde.org/p/bam/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@bam</a> As far as the "legacy syntax" is concerned: SPDX broke their conventions multiple times, so on the AppStream side I really got sick of these breaking changes and any syntax is fine. On the KDE side, switching the kaboutlicense keywords to SPDX is probably a bigger endeavour, and may not actually be worth the effort (I would guess that there may be some break risk here). So, this change looks fine to me now, as it is :-)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D6628">https://phabricator.kde.org/D6628</a></div></div><br /><div><strong>To: </strong>sebas, Plasma, sitter, mart, broulik<br /><strong>Cc: </strong>bam, mak, mart, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra<br /></div>