Review Request 122984: [kdeconnect] Make tests optional

[Andreas Sturmlechner]

> On March 17, 2015, 3:39 a.m., Albert Vaca Cintora wrote:
> > I've seen the discussion in the KIO package [1] and I have told my opinion there. I don't think we want the patch as it is now.
> > 
> > [1] https://git.reviewboard.kde.org/r/122206/

Thanks for your input! Actually what you describe is what I already had done for kdeconnect, only the kwin approach seemed to get more support even from those who were tendentially opposed to the idea. I would much prefer to get consensus on the matter.


- Andreas


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


On March 16, 2015, 9:59 p.m., Andreas Sturmlechner wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122984/
> -----------------------------------------------------------
> 
> (Updated March 16, 2015, 9:59 p.m.)
> 
> 
> Review request for kdeconnect.
> 
> 
> Repository: kdeconnect-kde
> 
> 
> Description
> -------
> 
> This patch makes it able to disable building tests by choice or unavailability of Qt5Test (by example of kwin).
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt b81a476a5fe749dcb22c2435747800cdede7c40e 
> 
> Diff: https://git.reviewboard.kde.org/r/122984/diff/
> 
> 
> Testing
> -------
> 
> Built with and without tests.
> 
> 
> Thanks,
> 
> Andreas Sturmlechner
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20150317/06e0509d/attachment.html>