[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-panel-devel
Subject:    D10920: Install app metadata for plasmoidviewer and plasmaengineexplorer
From:       Nathaniel Graham <noreply () phabricator ! kde ! org>
Date:       2018-02-28 21:00:39
Message-ID: 20180228210039.1.1C0D536E130BFA4B () phabricator ! kde ! org
[Download RAW message or body]

ngraham added a comment.


  Well the docs show you how an AppStream ID is to be constructed, and there's no \
mention of `.desktop`. I've had conversations with Matthias Klumpp, the AppStream \
maintainer, and he's indicated that my impression is accurate (and I believe he's \
said the validator will get much more strict in an upcoming major version).  
  Also I forgot to mention: since my proposed change will make the desktop file name \
no longer match the AppStream ID, you will also need to add:  
  `<launchable type="desktop-id">org.kde.plasmoidviewer.desktop</launchable>`

REPOSITORY
  R118 Plasma SDK

REVISION DETAIL
  https://phabricator.kde.org/D10920

To: kossebau, sebas, mart, broulik, ngraham
Cc: ngraham, plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, \
sebas, apol, mart


[Attachment #3 (unknown)]

<table><tr><td style="">ngraham 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/D10920" rel="noreferrer">View \
Revision</a></tr></table><br /><div><div><p>Well the docs show you how an AppStream \
ID is to be constructed, and there&#039;s no mention of <tt style="background: \
#ebebeb; font-size: 13px;">.desktop</tt>. I&#039;ve had conversations with Matthias \
Klumpp, the AppStream maintainer, and he&#039;s indicated that my impression is \
accurate (and I believe he&#039;s said the validator will get much more strict in an \
upcoming major version).</p>

<p>Also I forgot to mention: since my proposed change will make the desktop file name \
no longer match the AppStream ID, you will also need to add:</p>

<p><tt style="background: #ebebeb; font-size: 13px;">&lt;launchable \
type=&quot;desktop-id&quot;&gt;org.kde.plasmoidviewer.desktop&lt;/launchable&gt;</tt></p></div></div><br \
/><div><strong>REPOSITORY</strong><div><div>R118 Plasma SDK</div></div></div><br \
/><div><strong>REVISION DETAIL</strong><div><a \
href="https://phabricator.kde.org/D10920" \
rel="noreferrer">https://phabricator.kde.org/D10920</a></div></div><br \
/><div><strong>To: </strong>kossebau, sebas, mart, broulik, ngraham<br /><strong>Cc: \
</strong>ngraham, plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, \
abetts, sebas, apol, mart<br /></div>



[prev in list] [next in list] [prev in thread] [next in thread]