kossebau added a comment.

  In D27640#619817 <https://phabricator.kde.org/D27640#619817>, @cordlandwehr 
wrote:
  
  > My only question is, how is the X-KDE-PluginInfo-License value is 
evaluated? Are we free to write any value? Or do we have to extend some 
mechanism? I would be great to also use SPDX identifiers there!
  
  
  From what I can tell, Plasma itself currently ignores the license info, e.g. 
there is no display of About data for plasmoids or wallpaper plugins (sadly, 
IMHO, but no resources to fix myself).
  
  In general X-KDE-PluginInfo+* is feed to KPluginMetaData, which just holds 
the raw string. The string itself is evaluated e,g,in KAboutPluginDialog via 
KAboutLicense::byKeyword(), the only place I know about currently (besides 
deprecated KAboutData::fromPluginMetaData, which does the same 
KAboutLicense::byKeyword).
  
  KAboutLicense itself once was already extended to support keywords (see D6672 
<https://phabricator.kde.org/D6672>), but needs update to latest SPDX 
identifiers,

INLINE COMMENTS

> cordlandwehr wrote in metadata.desktop:96
> does this work with our PluginInfo mechanism? actually, I did not yet look 
> into the SPDX license handling in desktop files or any other tooling for 
> outbound licenses for libraries, plugins or applications; same question for 
> the same changes below in the desktop files

Hmpf, this is outdated version, forgot to append to commit before arc diff.
Should be "LGPLv21+" as I had found later that KAboutLicense still only 
supports the meanwhile deprecated spdx key variants.
Will update next.

REPOSITORY
  R242 Plasma Framework (Library)

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

To: kossebau, #plasma, cordlandwehr
Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns

Reply via email to