Look for kdesu in the correct location

Review Request #120185 - Created Sept. 13, 2014 and submitted

Information
Maarten De Meyer
kio
338755
120040
Reviewers
kdeframeworks
dfaure

kdesu is installed in libexec/ look for it there first. I left the findExecutable search as a backup.
Is looking in CMAKE_INSTALL_FULL_LIBEXECDIR correct? Or will kde-cli-tools be installed in libexec/kf5?

Insert 'kdesu' at the end to show a nicer error. If we leave this part out the error is "Could not launch 'root'" which is somewhat correct but not as easy to figure out as "Could not launch 'kdesu'"

Also added an unrelated QFile::decodeName() call.

Created .desktop file with X-KDE-SubstituteUID=true
Now I can launch it as root and when I remove kdesu I got a normal error message.

Unit test no longer skips because kdesu is not found.

Marco Martin
David Faure
Maarten De Meyer
Marco Martin
Sebastian Kügler
Maarten De Meyer
Review request changed

Status: Closed (submitted)

Loading...