<table><tr><td style="">heikobecker 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/D17650">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/D17650#379396" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D17650#379396</a>, <a href="https://phabricator.kde.org/p/habacker/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@habacker</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>I think this is unrelated - this request is to fix an issue with an available package on a distribution, so can anyone accept this ?</p></div>
</blockquote>

<p>I'd argue that the problem is with your distribution. Everybody still shipping qt4 is on its own anyway, seeing that it's not possible to upstream patches (and there are quite a few needed to keep it building) as the git repo is locked down. And I'm pretty sure this patch breaks at least one other framework. Additionally there might be future maintenance costs, so I see no reason to flog a dead horse and penalize everybody else for it.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D17650">https://phabricator.kde.org/D17650</a></div></div><br /><div><strong>To: </strong>habacker, Frameworks, lbeltrame, vonreth<br /><strong>Cc: </strong>heikobecker, ngraham, lbeltrame, kde-frameworks-devel, michaelh, bruns<br /></div>