D9888: Let kdesu work when __PATH_SU or __PATH_SUDO are not defined

[Milian Wolff]

mwolff created this revision.
mwolff added reviewers: dfaure, mpyne.
Restricted Application added a project: Frameworks.
Restricted Application added a subscriber: Frameworks.
mwolff requested review of this revision.

REVISION SUMMARY
  I quite frankly don't understand how this code can ever work. My
  system kdesu works fine, but a self-compiled one does not have either
  __PATH_SU nor __PATH_SUDO defined. Thus kdesu always starts
  /usr/bin/false which of course will always fail. You just get the
  super unhelpful "Su returned with an error" dialog shown...
  
  This patch only uses the two macros mentioned above when they are
  defined. If not, we try to launch su/sudo bei their name. This
  makes the code work for me.

REPOSITORY
  R299 KDESu

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D9888

AFFECTED FILES
  src/suprocess.cpp

To: mwolff, dfaure, mpyne
Cc: #frameworks
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180115/0f2ef9df/attachment.html>