D9207: Don't load KDE platform module in kglobalaccel5
Martin Flöser
noreply at phabricator.kde.org
Mon Dec 11 20:30:14 UTC 2017
graesslin added a comment.
In https://phabricator.kde.org/D9207#178257, @apol wrote:
> In https://phabricator.kde.org/D9207#176322, @graesslin wrote:
>
> > I'm against a risky change here. Especially as this becomes irrelevant with Wayland.
>
>
> What makes it risky?
The description: "I'm not as sure on this..." and "but from code skimming I can't see anything that
requires Plasma settings". So what I took from it is that it's unknown whether something might break. And that's what I consider risky. Especially in an application in frameworks which affects the stable Plasma releases. If kglobalaccel breaks all users are heavily affected. Due to that I'm extremely afraid of anything which could break.
REPOSITORY
R268 KGlobalAccel
REVISION DETAIL
https://phabricator.kde.org/D9207
To: hein, #plasma, davidedmundson, mart
Cc: apol, graesslin, plasma-devel, #frameworks, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20171211/67023d84/attachment.html>
More information about the Plasma-devel
mailing list