Bug 338755 - kdesu lookup broken
Summary: kdesu lookup broken
Status: RESOLVED FIXED
Alias: None
Product: frameworks-kio
Classification: Frameworks and Libraries
Component: general (show other bugs)
Version: 5.1.0
Platform: Other Linux
: NOR major
Target Milestone: ---
Assignee: David Faure
URL:
Keywords:
: 338756 (view as bug list)
Depends on:
Blocks:
 
Reported: 2014-09-02 18:18 UTC by Harald Sitter
Modified: 2015-05-18 18:37 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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/