<table><tr><td style="">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>FTR:</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">[13:35] <milian> how the hell can kdesu ever work? https://phabricator.kde.org/D9888 WTF
[13:35] <milian> does anyone know if I'm missing something?
[13:35] <milian> googling for these macros doesn't show me anything obvious either
[13:53] <sitter> milian: dead code I'd say. note the `if` following your change checks if the file 'false' is exectuable (which it isn't as that'd be ./false which likely never exists) and then falls back to QSP::findExec(cmd). where cmd is macro'd to sudo or su depending on the cmake switch KDESU_USE_SUDO_DEFAULT
[13:54] <milian> but it doesn't work for me without this patch
[13:54] <milian> so it cannot be dead code :D
[13:55] <milian> i.e. how can this work for anyone right now?
[13:58] <sitter> milian: http://paste.debian.net/1005279/ is how I read the original
[13:59] <milian> I'll see why this does not happen on my system
[14:02] <sitter> milian: try this for good measure http://paste.debian.net/1005280/
[14:02] <milian> yep
[14:03] <sitter> mind you, it could be that __PATH_SU/SUDO is actually defined somewhere in a system level include which would then make the QT_ACCESS pass and break things
[14:03] <milian> I can't find it in my  /usr/include at least
[14:06] <milian> sitter: QT_ACCESS("false", X_OK) == 0 on my system
[14:07] <milian> and no, there's no false in the current PWD
[14:07] <milian> it's in /usr/bin though
[14:07] <milian> I'd also be OK with removing that whole code and simplifying it
[14:08] <milian> like you proposed</pre></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R299 KDESu</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>To: </strong>mwolff, dfaure, mpyne<br /><strong>Cc: </strong>Frameworks<br /></div>