<table><tr><td style="">ngraham reopened this revision.<br />ngraham added a comment.<br />This revision is now accepted and ready to land.
</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/D17757">View Revision</a></tr></table><br /><div><div><p>This patch caused a critical regression on the stable branch: <a href="https://bugs.kde.org/show_bug.cgi?id=403117" class="remarkup-link" target="_blank" rel="noreferrer">https://bugs.kde.org/show_bug.cgi?id=403117</a>. This issue is affecting users on rolling release distros with KDE Applications 18.12.1.</p>

<p>I have reverted this patch on both the stable branch and master. Please feel free to give it another try, but this time please add an autotest for the failure condition in 403117 so we can make sure it doesn't regress again. Thanks!</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R319 Konsole</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D17757">https://phabricator.kde.org/D17757</a></div></div><br /><div><strong>To: </strong>pavelkh, Konsole, hindenburg<br /><strong>Cc: </strong>ngraham, konsole-devel, thsurrel, maximilianocuria, hindenburg<br /></div>