<table><tr><td style="">leinir added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D28701">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
color: #6B748C;
font-style: italic;
margin: 4px 0 12px 0;
padding: 8px 12px;
background-color: #F8F9FC;">
<div style="font-style: normal;
padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D28701#652898" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">D28701#652898</a>, <a href="https://phabricator.kde.org/p/ngraham/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@ngraham</a> wrote:</div>
<div style="margin: 0;
padding: 0;
border: 0;
color: rgb(107, 116, 140);"><p>Great. There are still a few more bugs though:</p>
<p>When you install certain global themes, they ask for authentication so install an SDDM theme. However when you uninstall that theme, it doesn't request authentication to remove them SDDM theme. So <tt style="background: #ebebeb; font-size: 13px;">/usr/share/sddm/themes</tt> accumulates a growing collection of unused themes:</p>
<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">ls /usr/share/sddm/themes/
breeze breeze-openSUSE elarun Layan maldives maya McMojave plasmaX Sweet</pre></div></div>
</blockquote>
<p>Not a huge amount knewstuff can do about that, that'll need to be done by the sddm kpackage plugin (mind you, having not looked i don't imagine this would be a huge issue, more a bit of forgotten implementation work in said plugin, since it already has the logic to ask on installation i don't imagine it would be a huge amount of effort to get it to do that on uninstallation as well).</p>
<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>When I install and uninstall the Sweet global theme, its Plasma theme still shows up in the Plasma style KCM. And looking in <tt style="background: #ebebeb; font-size: 13px;">~/.local/share/plasma/desktoptheme/</tt>, there are several orphaned plasma themes left over from global themes that I deleted from the GHNS dialog on the global themes KCM:</p>
<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">ls ~/.local/share/plasma/desktoptheme/
Arc-Dark kpluginindex.json Layan Sweet</pre></div>
<p>Their color schemes and icon themes are still there too.</p></blockquote>
<p>i can confirm this is happening, yup - however, this isn't knewstuff specifically, this is kpackage in general (you will notice this with any kpackage which has dependencies). I would suggest it is out of the scope for this specific patch to fix that problem, but it's definitely something we'll want to sort out (but also, if you have ever run apt-get purge or autoremove, that's the territory we're veering into here... that is to say, doable, but not trivial or automatically doable, as there's no guarantee that nothing else depends on that package being there).</p>
<p>Now, what we /could/ do is allow the user to just go "yup, please remove all the dependencies as well" for packages which have dependencies listed, but i am thinking that that is starting to sound quite dangerous... As in, what happens when a user goes "yes, i definitely know what i'm doing" and clicks that option, only to then discover that no, those dependencies were also needed by some other thing (i imagine a fair few global themes would have similar dependencies) and now that other thing just outright breaks when attempting to apply it. Again, this should hopefully be solvable, but i'm not sure how much work that would be. Arguably in scope here if we want to ask, but it's not a regression (by which i mean kpackage -r Sweet -t Plasma/LookAndFeel would also leave those other bits behind).</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R304 KNewStuff</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D28701">https://phabricator.kde.org/D28701</a></div></div><br /><div><strong>To: </strong>leinir, Plasma, KNewStuff, Frameworks, ngraham, mart, davidedmundson, broulik, bshah<br /><strong>Cc: </strong>ngraham, kde-frameworks-devel, LeGast00n, cblack, michaelh, bruns<br /></div>