Re: Review Request 120040: Install kdesu under bin

2015-01-29 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120040/ --- (Updated Jan. 29, 2015, 3:16 p.m.) Status -- This change has been

Re: Review Request 120040: Install kdesu under bin

2014-09-16 Thread Marco Martin
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

Re: Review Request 120040: Install kdesu under bin

2014-09-16 Thread Marco Martin
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... right, it seems on both places here, but is probably a distro thing. Any idea

Re: Review Request 120040: Install kdesu under bin

2014-09-16 Thread 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

Re: Review Request 120040: Install kdesu under bin

2014-09-16 Thread Maarten De Meyer
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

Re: Review Request 120040: Install kdesu under bin

2014-09-05 Thread 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

Review Request 120040: Install kdesu under bin

2014-09-02 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120040/ --- Review request for KDE Frameworks and Plasma. Repository: kde-cli-tools

Re: Review Request 120040: Install kdesu under bin

2014-09-02 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120040/ --- (Updated Sept. 2, 2014, 6:26 p.m.) Review request for KDE Frameworks and

Re: Review Request 120040: Install kdesu under bin

2014-09-02 Thread Marco Martin
--- 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.

Re: Review Request 120040: Install kdesu under bin

2014-09-02 Thread Marco Martin
--- 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.

Re: Review Request 120040: Install kdesu under bin

2014-09-02 Thread Marco Martin
--- 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

Re: Review Request 120040: Install kdesu under bin

2014-09-02 Thread Harald Sitter
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120040/#review65732 --- doesn't actually affect the second bug, if one were to write

Re: Review Request 120040: Install kdesu under bin

2014-09-02 Thread Lukáš Tinkl
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120040/#review65734 --- kdesu/CMakeLists.txt

Re: Review Request 120040: Install kdesu under bin

2014-09-02 Thread Hrvoje Senjan
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120040/#review65738 --- in kde4 kdesu was installed under bin it was actually also

Re: Review Request 120040: Install kdesu under bin

2014-09-02 Thread Marco Martin
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 :/ 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