----------------------------------------------------------- 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 > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
