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