Review Request 110870: Add Optional Accounts-SSO support to ktp-auth-handler

[David Edmundson]

> On June 7, 2013, 5:14 p.m., Daniele E. Domenichelli wrote:
> > CMakeLists.txt, lines 18-19
> > <http://git.reviewboard.kde.org/r/110870/diff/1/?file=149002#file149002line18>
> >
> >     since these are optional, perhaps use macro_optional_find_package?
> 
> Aleix Pol Gonzalez wrote:
>     I wouldn't do that. It's only use is to explicitly disable the finding from the cache.
>     
>     That said, if it was me, I'd make them REQUIRED.

Being able to enable/disable independently would be a bad thing. AccountsQt without SignOnQt or vice versa would be really bad, so we would have to manually write our own combo of OPTION and FIND_PACKAGE.. and that's probably too much effort for the reward.

Adding REQUIRED would be really bad as that half our distributions wouldn't be able to compile and everyone would be stuck on 0.6 forever. 

I'm fine with this as-is.


- David


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/110870/#review33908
-----------------------------------------------------------


On June 7, 2013, 5:19 p.m., Àlex Fiestas wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/110870/
> -----------------------------------------------------------
> 
> (Updated June 7, 2013, 5:19 p.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Description
> -------
> 
> Adds Accounts-SSO support to ktp-auth-handler by checking if the account has a StorageIdentifier (which we assume comes from Accounts) and X-FACEBOOK-PLATFORM mechanism exsts.
> 
> We'll probably have to implement something special for Google as well.
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt 6c51903 
>   cmake/modules/FindAccountsQt.cmake PRE-CREATION 
>   cmake/modules/FindSignOnQt.cmake PRE-CREATION 
>   getcredentialsjob.h PRE-CREATION 
>   getcredentialsjob.cpp PRE-CREATION 
>   sasl-auth-op.h 6e0e5ef 
>   sasl-auth-op.cpp b745234 
>   x-telepathy-sso-operation.h PRE-CREATION 
>   x-telepathy-sso-operation.cpp PRE-CREATION 
> 
> Diff: http://git.reviewboard.kde.org/r/110870/diff/
> 
> 
> Testing
> -------
> 
> Tested with and without support, it builds and works in both cases.
> 
> 
> Thanks,
> 
> Àlex Fiestas
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20130607/9f7c90f8/attachment-0001.html>