<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#653954" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D28701#653954</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>I mean, GHNS basically *is* a userspace package manager, and things like dependency management and removal of stale files are why package managers exist. :)</p>

<p>If it needs to be fixed/implemented in other patches, that's okay.</p></div>
</blockquote>

<p>Definitely, yes, it wants implementing, though i think we need to consider precisely where we want it to go... Thinking it probably wants to go in KPackage itself for this case (since that's what installs those dependencies - it's at the end of the packagejobthread's installPackage function, but i'm thinking the optional removal thing likely needs to go into the packagejob, possibly as an overload... just brainfarting a bit here), but there's also a dependencies and referencing type thing in OCS (and consequently KNewStuff) which we'll need to implement more... properly than it is right now. But yup, that's sort of orthogonal to this patch anyway, more a "this wants to be done" type thing :)</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>