<div dir="ltr">On Fri, Sep 13, 2013 at 1:17 PM, 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="HOEnZb"><div class="h5">On Friday 13 September 2013 13:03:29 Aleix Pol wrote:<br>
> On Fri, Sep 13, 2013 at 9:43 AM, David Faure <<a href="mailto:faure@kde.org">faure@kde.org</a>> wrote:<br>
> > 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<br>
> ><br>
> > once<br>
> ><br>
> > > in the root CMakeLists.txt. Actually this should be the only file with<br>
> > > "find_package" calls.<br>
> ><br>
> > 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>
> ><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>
> Well, then let's make this a especial case for autotests/ and tests/. We<br>
> shouldn't have any find_package() within src/, and that's the case in many<br>
> places.<br>
<br>
</div></div>Not tests/. That's for manual tests, no qtestlib there.<br>
<div class="HOEnZb"><div class="h5"><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>
</div></div></blockquote></div></div><div class="gmail_extra"><br></div><div class="gmail_extra">Well, but in that case you'll maybe want to find QtGui, QtWidgets or KF5::AwesomeTests.</div><div class="gmail_extra"><br>

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