Create Task
Maniphest T15473

Port KRunner DBus runner loading away from deprecated KPluginMetaData::fromDesktopFile
Open, Needs TriagePublic
Actions

Description

This blocks building with deprecation level 5.92 or above, which in turn is needed for enabling CI on kde-cli-tools.

vkrause created this task.Apr 24 2022, 4:05 PM
alex renamed this task from Port KRunner unit tests away from deprecated KPluginMetaData::fromDesktopFile to Port KRunner DBus runner loading away from deprecated KPluginMetaData::fromDesktopFile.Apr 29 2022, 4:38 PM
alex added a subscriber: alex.Apr 29 2022, 4:57 PM

Unlike other plugin systems, we want to keep the desktop files compatible for KF6.

alex claimed this task.Apr 29 2022, 5:19 PM
alex added a comment.Apr 30 2022, 7:21 AM

https://invent.kde.org/frameworks/krunner/-/merge_requests/97 is a draft of mine. Though I am not sure if we want to keep the X-KDE-PluginInfo prefix when reading the author information. IMHO it is rather ugly and represents a terminology we no longer have with us getting rid of KPluginInfo.

Generally speaking, I am asking myself if the X-Plasma-Runner prefix is a good idea. My preference would be X-KRunner as a prefix and then add some compat logic to convert the old-style entries at runtime.

alex added a comment.May 21 2022, 4:05 PM

i have done the porting while noch changing the keys. generally speaking, we should have a discussion about the prefixes in KF6, since other frameworks are also affected.

alex moved this task from Backlog to Done on the KF6 board.May 21 2022, 4:08 PM