<table><tr><td style="">mwolff updated this revision to Diff 25395.<br />mwolff retitled this revision from "Let kdesu work when __PATH_SU or __PATH_SUDO are not defined" to "Make kdesu work when PWD is /usr/bin".<br />mwolff edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-ahpqqbbvyn5eold/" rel="noreferrer">(Show Details)</a><br />mwolff added a subscriber: sitter.<br />mwolff 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/D9888" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>cleanup now that I figured out what happens, thanks <a href="https://phabricator.kde.org/p/sitter/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@sitter</a></p></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">I quite frankly don't understand how this code can </span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">This code seems to have accumulated quite some legacy cruft. The<br />
two defines are n</span>ever <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">work.</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">set from CMake,</span> <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">My</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">thus they get defined to</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">system kdesu works fine</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">"false". Now when the PWD contains a file called "false"</span>, <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">but a self-compiled one does not have either</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">like is</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">`__PATH_SU` nor `__PATH_SUDO` defined.</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">usually the case for /usr/bin,</span> <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Thus</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">then</span> kdesu <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">always starts</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">would suddenly stop working.</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">/usr/bin/false which of course will always fail. You just get the</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">You just get the super unhelpful "Su returned with an error" dialog</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">super unhelpful "Su returned with an error" dialog </span>shown...<br />
<br />
This patch <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">only us</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">remov</span>es the <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">two</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">obsolete</span> macros <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">mentioned above when they are</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">and always uses QStandardPath</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">defined. If not, we try to launch su/sudo bei their name</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">to find the executable path for the superUserCommand</span>. This<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"> makes the</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">makes</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">code work for me when</span> the <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">code work for me</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">PWD contains a file called "false"</span>.</div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R299 KDESu</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D9888?vs=25392&id=25395" rel="noreferrer">https://phabricator.kde.org/D9888?vs=25392&id=25395</a></div></div><br /><div><strong>BRANCH</strong><div><div>master</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D9888" rel="noreferrer">https://phabricator.kde.org/D9888</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/suprocess.cpp</div></div></div><br /><div><strong>To: </strong>mwolff, dfaure, mpyne<br /><strong>Cc: </strong>sitter, Frameworks<br /></div>