D23469: Move the notifier logic into a separate process

Aleix Pol Gonzalez noreply at phabricator.kde.org
Mon Aug 26 17:40:43 BST 2019


apol created this revision.
apol added a reviewer: Plasma.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
apol requested review of this revision.

REVISION SUMMARY
  At the moment, the bit that notifies about system updates was a plasmoid and it was working well. The problem was that as a plasmoid it did pull quite a lot of dependencies (flatpak, packagekit) that we kept alive and sometimes were found to de-stabilize the plasmashell process.
  This patch moves this into a SNI, the UX stays the same, if it crashes, only this will crash.

TEST PLAN
  Been using it, seems to work okay.

REPOSITORY
  R134 Discover Software Store

BRANCH
  notifier

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

AFFECTED FILES
  discover/CMakeLists.txt
  notifier/CMakeLists.txt
  notifier/DiscoverNotifierDeclarativePlugin.cpp
  notifier/DiscoverNotifierDeclarativePlugin.h
  notifier/main.cpp
  notifier/org.kde.discover.notifier.desktop.cmake
  notifier/plasmoid/Messages.sh
  notifier/plasmoid/contents/ui/main.qml
  notifier/plasmoid/metadata.desktop
  notifier/qmldir

To: apol, #plasma
Cc: plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190826/28790754/attachment.html>


More information about the Plasma-devel mailing list