Bug 338755

Summary: kdesu lookup broken
Product: [Frameworks and Libraries] frameworks-kio Reporter: Harald Sitter <sitter>
Component: generalAssignee: David Faure <faure>
Status: RESOLVED FIXED    
Severity: major CC: de.meyer.maarten, kdelibs-bugs, rdieter, simonandric5
Priority: NOR    
Version: 5.1.0   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:

Description Harald Sitter 2014-09-02 18:18:00 UTC
in desktopexecparser.cpp there's this line:
            result << QStandardPaths::findExecutable("kdesu") << "-u";

which fails to find kdesu because that is installed to prefix/lib/libexec (kdesu is part of kde-cli-tools)

this breaks *all* desktop files with X-KDE-SubstituteUID=true as desktopexecparser is used by klauncher to construct the launch command for kdeinit and since it returns an empty path where kdesu is supposed to go it will try to launch something like "" "-u root /usr/bin/kittens"


Reproducible: Always

Steps to Reproduce:
1. add X-KDE-SubstituteUID=true to any desktop file
2. kbuildsycoca5
3. try to launch the app via plasma/krunner

Actual Results:  
completely useless and bugged out error window

Expected Results:  
kdesu should come up
Comment 1 Maarten De Meyer 2014-09-13 15:59:24 UTC
*** Bug 338756 has been marked as a duplicate of this bug. ***
Comment 2 Maarten De Meyer 2014-09-13 15:59:56 UTC
https://git.reviewboard.kde.org/r/120185/