<div dir="ltr">On Fri, Sep 13, 2013 at 9:43 AM, David Faure <span dir="ltr"><<a href="mailto:faure@kde.org" target="_blank">faure@kde.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im">On Thursday 12 September 2013 02:32:20 Aleix Pol wrote:<br>
> I'd say that all Qt dependencies in the module should be defined only once<br>
> in the root CMakeLists.txt. Actually this should be the only file with<br>
> "find_package" calls.<br>
<br>
</div>And I disagree. If you build the framework with unittests disabled (like<br>
distros will probably do) then there's no point in searching for QtTest.<br>
Modularity wins again, better do that inside the autotests subdir.<br>
<br>
Apart from QtTest, I agree, though.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
David Faure, <a href="mailto:faure@kde.org">faure@kde.org</a>, <a href="http://www.davidfaure.fr" target="_blank">http://www.davidfaure.fr</a><br>
Working on KDE, in particular KDE Frameworks 5<br>
<br>
</font></span></blockquote></div><br></div><div class="gmail_extra">Well, then let's make this a especial case for autotests/ and tests/. We shouldn't have any find_package() within src/, and that's the case in many places.</div>

<div class="gmail_extra"><br></div><div class="gmail_extra">Aleix</div></div>