Review Request 122606: Fix startup of kssld5 dbus service

[David Faure]

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

Ship it!


Ah. kiod registers as kssld5, but if it's not running then this doesn't work indeed.

We could install a .service file for kssld but ok, let's do this for now. My idea of making this independent from the process actually hosting kssld is more trouble than I thought.

- David Faure


On March 1, 2015, 2:29 a.m., Bartosz Sławianowski wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122606/
> -----------------------------------------------------------
> 
> (Updated March 1, 2015, 2:29 a.m.)
> 
> 
> Review request for KDE Frameworks and David Faure.
> 
> 
> Repository: kio
> 
> 
> Description
> -------
> 
> KSslCertificateManager is trying to connect to org.kde.kssld5 directly which doesn't work because it lacks it's own .service dbus file needed for start-on-demand.
> 
> This patch makes it connect correctly to org.kde.kiod5 (which then loads kssld5 module and makes service available) instead.
> 
> 
> Diffs
> -----
> 
>   src/core/ksslcertificatemanager.cpp a47ca68 
> 
> Diff: https://git.reviewboard.kde.org/r/122606/diff/
> 
> 
> Testing
> -------
> 
> KIO git with patch, tested with Konversation which used to spam SSL errors before applying patch.
> 
> 
> Thanks,
> 
> Bartosz Sławianowski
> 
>

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