> On Jan. 29, 2015, 3:01 p.m., Marco Martin wrote: > > any updates on that? > > > > here the patch works correctly after i make kdesu to be installed under > > CMAKE_INSTALL_FULL_LIBEXECDIR_KF5 instead of CMAKE_INSTALL_FULL_LIBEXECDIR > > > > I would push this asap, since kdesu in Plasma 5 is *still* broken. > > if CMAKE_INSTALL_FULL_LIBEXECDIR_KF5 is ok (that seems where it's the > > safest from a coinstallability pov), i'll move where kdesu is installed as > > well
as in, would still need http://paste.opensuse.org/67799408 - Marco ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120185/#review74983 ----------------------------------------------------------- On Oct. 4, 2014, 3:32 p.m., Maarten De Meyer wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/120185/ > ----------------------------------------------------------- > > (Updated Oct. 4, 2014, 3:32 p.m.) > > > Review request for KDE Frameworks and David Faure. > > > Bugs: 338755 > https://bugs.kde.org/show_bug.cgi?id=338755 > > > Repository: kio > > > Description > ------- > > kdesu is installed in libexec/ look for it there first. I left the > findExecutable search as a backup. > Is looking in CMAKE_INSTALL_FULL_LIBEXECDIR correct? Or will kde-cli-tools be > installed in libexec/kf5? > > Insert 'kdesu' at the end to show a nicer error. If we leave this part out > the error is "Could not launch 'root'" which is somewhat correct but not as > easy to figure out as "Could not launch 'kdesu'" > > Also added an unrelated QFile::decodeName() call. > > > Diffs > ----- > > autotests/krununittest.cpp b1da9aa > src/core/desktopexecparser.cpp 9510697 > > Diff: https://git.reviewboard.kde.org/r/120185/diff/ > > > Testing > ------- > > Created .desktop file with X-KDE-SubstituteUID=true > Now I can launch it as root and when I remove kdesu I got a normal error > message. > > Unit test no longer skips because kdesu is not found. > > > Thanks, > > Maarten De Meyer > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
