----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106515/#review19239 -----------------------------------------------------------
This review has been submitted with commit 900824a2f8ea5eb35a483e40498a593842bdb7ce by David Edmundson to branch kde-telepathy-0.5. - Commit Hook On Sept. 20, 2012, 4:38 p.m., David Edmundson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/106515/ > ----------------------------------------------------------- > > (Updated Sept. 20, 2012, 4:38 p.m.) > > > Review request for Telepathy. > > > Description > ------- > > Check connection manager supports protocol listed in profile. This should fix > the issue of skype appearing in the list when telpeathy-haze was installed > but pidgin-skype was not. > > ConnectionManagers require having becomeReady() called before the list of > protocols they support can be used, hence converting from a list of CMs to a > hash of CmPtrs > > Relevant IRC conversation about ABI breaking: > [16:53] <d_ed> gkiagia: how bad is it to break the ABI on a private library > with no installed headers that no-one else uses between 0.5.0 and 0.5.1 > [16:53] <d_ed> (the accounts KCM lib) > [16:53] <gkiagia> not an issue > > > Diffs > ----- > > src/KCMTelepathyAccounts/profile-list-model.cpp > 35ced7cb3e4b4959a798ac6a7a7ff7369f10bb6e > src/KCMTelepathyAccounts/profile-list-model.h > d133ffcffec376064264c4310e39d6e0f0f836d0 > > Diff: http://git.reviewboard.kde.org/r/106515/diff/ > > > Testing > ------- > > > Thanks, > > David Edmundson > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
