D11126: [touchpad kded] Work around kded fail on X-KDE-OnlyShowOnQtPlatforms lists

[Friedrich W . H . Kossebau]

This revision was automatically updated to reflect the committed changes.
Closed by commit R119:830ff8deb477: [touchpad kded] Work around kded fail on 
X-KDE-OnlyShowOnQtPlatforms lists (authored by kossebau).

REPOSITORY
  R119 Plasma Desktop

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D11126?vs=28934=30069

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

AFFECTED FILES
  kcms/touchpad/src/kded/kded_touchpad.desktop

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

D11126: [touchpad kded] Work around kded fail on X-KDE-OnlyShowOnQtPlatforms lists

[Friedrich W . H . Kossebau]

kossebau added a comment.


  I am slightly confused though as for me kded still seems to load the kded 
module anyway, as the current code in kded only checks the platform on 
preloading, but not later. Is that really on purpose?

REPOSITORY
  R119 Plasma Desktop

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

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

D11126: [touchpad kded] Work around kded fail on X-KDE-OnlyShowOnQtPlatforms lists

[Friedrich W . H . Kossebau]

kossebau created this revision.
kossebau added a reviewer: mart.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
kossebau requested review of this revision.

REVISION SUMMARY
  kded currently fails to handle list values set for X-KDE-OnlyShowOnQtPlatforms
  and will read the whole value as one string "xcb;" including the semicolon.

REPOSITORY
  R119 Plasma Desktop

BRANCH
  fixOnlyShowOnQtPlatforms

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

AFFECTED FILES
  kcms/touchpad/src/kded/kded_touchpad.desktop

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