Summary: | Empty plugin directories break Plasma Shell | ||
---|---|---|---|
Product: | [Plasma] plasmashell | Reporter: | Pau Garcia i Quiles <pgquiles> |
Component: | general | Assignee: | David Edmundson <kde> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | bhush94, plasma-bugs |
Priority: | NOR | ||
Version: | 5.3.0 | ||
Target Milestone: | 1.0 | ||
Platform: | Ubuntu | ||
OS: | Linux | ||
Latest Commit: | http://commits.kde.org/plasma-workspace/f59164914c46eca3686a3ade63f46bf6489f9193 | Version Fixed In: |
Description
Pau Garcia i Quiles
2015-05-21 12:42:48 UTC
I did not explicitly say it but the moment I deleted the directories listed in step 4, plasmashell no longer looked for shell handlers under /usr/local and found Plasma Desktop under /usr. Problem solved. It's not a critical bug if it only affects people with a slightly weird setup. As for the bug, we're currently only loading and storing the one directory. Given potentially I could install plasma-workspace from packages and compile plasma-media-center into /whatever, we should check all dirs static const QString s_shellsDir(QStandardPaths::locate(QStandardPaths::QStandardPaths::GenericDataLocation, PLASMA_RELATIVE_DATA_INSTALL_DIR "/shells/", QStandardPaths::LocateDirectory)); I'll change this to a locateAll and a list. Git commit f59164914c46eca3686a3ade63f46bf6489f9193 by David Edmundson. Committed on 21/05/2015 at 15:42. Pushed by davidedmundson into branch 'master'. check all shell directories REVIEW: 123870 M +20 -18 shell/shellmanager.cpp http://commits.kde.org/plasma-workspace/f59164914c46eca3686a3ade63f46bf6489f9193 |