<table><tr><td style="">lbeltrame added a subscriber: plasma-devel.<br />lbeltrame added a comment.</td></tr></table><br /><div><div><p>Added as of <a href="https://phabricator.kde.org/rBREEZEGTK4fcde887a817e948c1605d42091ca11504367f43" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">4fcde887a817e948c1605d42091ca11504367f43</a>.</p>

<p>But seriously, why didn't this come up in the initial review? Like the libtaskmanager patch, did anyone try to actually build and install what is being reviewed, aside checking the code?</p>

<p>Yes, a bit ticked off, even though it's my fault originally in this specific revision, and I fully acknowledge that.</p></div></div><br /><div><strong>BRANCHES</strong><div><div>master</div></div></div><br /><div><strong>USERS</strong><div><div>lbeltrame (Author)</div></div></div><br /><div><strong>COMMIT</strong><div><a href="https://phabricator.kde.org/rBREEZEGTK93eb6e6ebcb8" rel="noreferrer">https://phabricator.kde.org/rBREEZEGTK93eb6e6ebcb8</a></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>lbeltrame<br /><strong>Cc: </strong>plasma-devel, dirrukd<br /></div>