<table><tr><td style="">ngraham created this revision.<br />ngraham added reviewers: Kate, dhaumann, cullmann.<br />Restricted Application added a project: Kate.<br />Restricted Application added a subscriber: kwrite-devel.<br />ngraham requested review of this revision.
</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/D13138">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>The original change (<a href="https://phabricator.kde.org/R40:9adcebd3c2e476c8a32e9b455cc99f46b0e12a7e" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">9adcebd3c2e476c8a32e9b455cc99f46b0e12a7e</a>) to prevent sudo usage broke the use case of running KWrite or Kate while logged in as the actual <tt style="background: #ebebeb; font-size: 13px;">root</tt> user with a GUI session. This is how the Kali distro is set up by default, so the original change amounted to making Kate and KWrite not launch at all on this KDE distro.</p>
<p>This patch re-enables running as the actual root user, but keeps blocking usage via <tt style="background: #ebebeb; font-size: 13px;">sudo</tt> or <tt style="background: #ebebeb; font-size: 13px;">kdesu</tt>.</p>
<p>I have submitted a similar change for Dolphin that has been accepted (<a href="https://phabricator.kde.org/D12795" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">D12795</a>), but <a href="https://phabricator.kde.org/p/elvisangelaccio/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@elvisangelaccio</a> wants that to go in at the same time as this, to keep them in sync.</p>
<p>BUG: 387973<br />
FIXED-IN: 18.08.0</p></div></div><br /><div><strong>TEST PLAN</strong><div><ul class="remarkup-list">
<li class="remarkup-list-item">Log in as normal user and run <tt style="background: #ebebeb; font-size: 13px;">sudo kate</tt> or <tt style="background: #ebebeb; font-size: 13px;">sudo kwrite</tt>: you get an error message.</li>
<li class="remarkup-list-item">Log in as normal user and run <tt style="background: #ebebeb; font-size: 13px;">kdesu kate</tt> or <tt style="background: #ebebeb; font-size: 13px;">kdesu kwrite</tt>: you get an error message.</li>
<li class="remarkup-list-item">Log in as the root user and run Kate or KWrite normally: it works.</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R40 Kate</div></div></div><br /><div><strong>BRANCH</strong><div><div>allow-running-as-the-actual-root-user (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D13138">https://phabricator.kde.org/D13138</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>kate/main.cpp<br />
kwrite/main.cpp</div></div></div><br /><div><strong>To: </strong>ngraham, Kate, dhaumann, cullmann<br /><strong>Cc: </strong>kwrite-devel, elvisangelaccio, michaelh, kevinapavew, ngraham, demsking, cullmann, sars, dhaumann<br /></div>