Review Request 120040: Install kdesu under bin

[David Faure]

On Sept. 2, 2014, 6:37 p.m., Marco Martin wrote:
> > I'm a bit worried by the multitude of user and distro specific scripts that rely on kdesu being present :/
> 
> Marco Martin wrote:
>     to me either way it gets fixed i'm ok.
>     the other option is to keep it called kdesu, so it wouldn't be coinstallable so distributions would need to make kdesu a package of its own, alternative with the kde4 kdesu, but it may look even more problematic to me

Scripts will just have to be updated, just like scripts calling kioclient4, kde4-config, kstart, kdecp, kbuildsycoca4 and so many other things.

We can't have both source-compat-for-scripts and co-installability. It's one or the other.


- David


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


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/20140905/6ccee4bf/attachment.html>