Review Request 120040: Install kdesu under bin

[Harald Sitter]

> On Sept. 2, 2014, 6:54 p.m., Hrvoje Senjan wrote:
> > >in kde4 kdesu was installed under bin
> > 
> > it was actually also in libexec. just that KStandardDirs::findExe() looked in libexec paths, QStandardPaths doesn't...
> 
> Marco Martin wrote:
>     right, it seems on both places here, but is probably a distro thing.
>     Any idea of a way to find stuff under libexec on kf5?
>     in other places the path is burned in in a define set by cmake, but that's probably not flexible enough

Actually it appears to me that it wasn't in bin for kde4, some distros just worked around that because users where whining.

That being said, I think at this point moving it bin would be counterproductive as users really should use pkexec as it is cross-desktop, cross-distro (mind kdesudo on kubuntu for example) and provides meaningful workspace integration through the relevant polkit-agent gui. In fact at the kubuntu/debian bof at akademy we were talking about proposing exactly that for all our usage of kdesu as to possibly retry the entire bugger in favor of pkexec with kf6.


- Harald


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/120040/#review65738
-----------------------------------------------------------


On Sept. 2, 2014, 6:27 p.m., Marco Martin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/120040/
> -----------------------------------------------------------
> 
> (Updated Sept. 2, 2014, 6:27 p.m.)
> 
> 
> Review request for KDE Frameworks and Plasma.
> 
> 
> Repository: kde-cli-tools
> 
> 
> Description
> -------
> 
> this is a part of adressing the bug
> https://bugs.kde.org/show_bug.cgi?id=338755
> https://bugs.kde.org/show_bug.cgi?id=338756
> 
> in kde4 kdesu was installed under bin, and should still, being something that should be invokable 
> but renames it to kdesu5, for coinstallability reasons
> 
> therefore, kio/src/core/desktopexecparser.cpp should be modified to search for kdesu5 instead of kdesu
> 
> 
> Diffs
> -----
> 
>   kdesu/CMakeLists.txt 2a70831 
> 
> Diff: https://git.reviewboard.kde.org/r/120040/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Marco Martin
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140916/13a6343a/attachment.html>