> On Jan. 23, 2015, 8:43 a.m., Vishesh Handa wrote:
> > I'm not against this, but I am curious as to why this is being done. 
> > 
> > I would think that packagers should be building the tests and running them 
> > on their platform and make sure everything passes. We have a strict policy 
> > that all tests must always pass.

This is mostly useful on source based distributions (specifically, this patch 
comes from Gentoo).  While in general running tests everywhere would be great, 
source distro users may not have the cpu time to compile/run tests.  Also, some 
test suites don't work and users may not care to figure out why (for instance, 
last time I tried enabling tests in Gentoo, binutils failed its suite).

For binary distriubtions, I agree they should be running tests (especially 
since we work to keep them green).  But source based distros aren't so clear 
cut.


- Matthew


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


On Jan. 22, 2015, 2:48 p.m., Andreas Sturmlechner wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122206/
> -----------------------------------------------------------
> 
> (Updated Jan. 22, 2015, 2:48 p.m.)
> 
> 
> Review request for KDE Frameworks.
> 
> 
> Repository: kio
> 
> 
> Description
> -------
> 
> [kio] Make tests optional
> This is a small patch to CMakeLists.txt to only depend on Qt5Test if 
> BUILD_TESTING.
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt 7fe0be5d4b2d7d9475a7844b4f8d93fc2f0a00c3 
> 
> Diff: https://git.reviewboard.kde.org/r/122206/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Andreas Sturmlechner
> 
>

_______________________________________________
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Reply via email to