Review Request 116935: Lookup paths for kioslaves in KLauncher

Alex Merry alex.merry at kde.org
Sat Mar 22 14:58:31 UTC 2014


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

(Updated March 22, 2014, 2:58 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Frameworks, David Faure and Hrvoje Senjan.


Repository: kinit


Description
-------

Lookup paths for kioslaves in KLauncher

If you pass kdeinit anything other than an absolute path, it assumes it
is a normal kdeinit executable - ie, it is an executable <name> in
$PATH, optionally with a libkdeinit5_<name> in a library path.

kioslaves are actually installed as plugins, though, so we do the lookup
in klauncher's requestSlave() function and pass an absolute path to
kdeinit.

Fix up kioslave debug and valgrind modes

valgrind mode with kdeinit still requires the slave module as an
argument, and debug mode cannot work without kdeinit.


Diffs
-----

  src/klauncher/klauncher.cpp a8630854af4bd3094b9688c3f9a40d10516d2056 

Diff: https://git.reviewboard.kde.org/r/116935/diff/


Testing
-------

Built, installed.
Killed all existing kdeinit5/klauncher/etc instances.
Ran kdeinit5 on one terminal.
Ran kioslavetest from kio/tests on another terminal.
Used it to list /tmp.
kioslavetest's terminal printed the contents of /tmp, and kdeinit5's terminal printed

kdeinit5: Got EXEC_NEW '/home/kf5-devel/kf5/lib64/plugins/kf5/kio_file.so' from launcher.
kdeinit5: preparing to launch '/home/kf5-devel/kf5/lib64/plugins/kf5/kio_file.so'
kio_file.so(23605)/(default) [31mQStandardPaths::writableLocation[0m: QStandardPaths: XDG_RUNTIME_DIR not set, defaulting to '/tmp/runtime-kf5-devel'
kdeinit5: PID 23605 terminated.


Thanks,

Alex Merry

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140322/b71fea00/attachment.html>


More information about the Kde-frameworks-devel mailing list