<div dir="auto"><div><div class="gmail_extra"><div class="gmail_quote">On 30/08/2017 4:10 PM, "Friedrich W. H. Kossebau" <<a href="mailto:kossebau@kde.org">kossebau@kde.org</a>> wrote:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Ben & all,<br></blockquote></div></div></div><div dir="auto"><br></div><div dir="auto">Hi Friedrich,</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="quoted-text"><br>
Am Samstag, 19. August 2017, 02:24:21 CEST schrieb Ben Cooksley:<br>
> On Wed, Aug 16, 2017 at 3:17 AM, David Faure <<a href="mailto:faure@kde.org">faure@kde.org</a>> wrote:<br>
> > Hi everyone,<br>
><br>
> Hi David,<br>
><br>
> > The documentation for how to do that is now up at<br>
> ><br>
> >  <a href="https://blogs.kde.org/2017/08/15/running-applications-and-unittests-witho" rel="noreferrer" target="_blank">https://blogs.kde.org/2017/08/<wbr>15/running-applications-and-<wbr>unittests-witho</a><br>
> >  ut-make-install><br>
> > I have ported/fixed all frameworks, except for kirigami, and there are<br>
> > still pending merge requests for kpackage.<br>
> ><br>
> > I'm wondering if we could change the CI scripts so that they run the<br>
tests<br>
> > before doing make install? For frameworks only, for now.<br>
><br>
> I've now done this, for Linux at least.<br>
<br>
</div>Any chance the bits about "For frameworks only, for now." was missed here?<br></blockquote></div></div></div><div dir="auto"><br></div><div dir="auto">I applied it to the XenialQt5.7 platform, because at the moment to my knowledge only Frameworks uses that.</div><div dir="auto"><br></div><div dir="auto">It could be however that KDevelop is using it too though.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
E.g. for kdevelop stable/master builds I now also see tests been run before<br>
install happens (which fails many of them).<br>
<br>
Seems to me those bits were important, if looking at the code:<br>
<br>
that code in ECM to support running tests uninstalled right now is only<br>
triggered by some "find_package(ECM x.y.z)", with x.y.z >= 5.38.0<br>
<br>
Cmp. the<br>
        NOT ("${ECM_GLOBAL_FIND_VERSION}" VERSION_LESS "5.38.0")<br>
in <a href="https://cgit.kde.org/extra-cmake-modules.git/tree/kde-modules/
KDECMakeSettings.cmake#n252" rel="noreferrer" target="_blank">https://cgit.kde.org/extra-<wbr>cmake-modules.git/tree/kde-<wbr>modules/<br>
KDECMakeSettings.cmake#n252</a> , where ${ECM_GLOBAL_FIND_VERSION} is from<br>
        set(ECM_GLOBAL_FIND_VERSION "${ECM_FIND_VERSION}")<br>
in <a href="https://cgit.kde.org/extra-cmake-modules.git/tree/ECMConfig.cmake.in#n11" rel="noreferrer" target="_blank">https://cgit.kde.org/extra-<wbr>cmake-modules.git/tree/<wbr>ECMConfig.cmake.in#n11</a><br>
<br>
And that find_package of ECM with 5.38.0 is something only the KDE Frameworks<br>
repos do right now. Where other projects, like kdevelop, are fine with less<br>
recent versions (e.g. kdevelop master only expects ECM 5.14.0 as minimum),<br>
and thus would not trigger that new build layout needed for uninstalled<br>
testing.<br>
<br>
So is it possible to limit that new build steps order to just the Frameworks<br>
product, and restore old behaviour for anyone else?<br></blockquote></div></div></div><div dir="auto"><br></div><div dir="auto">We'd need to change code in the DSL scripts and supply an alternative pipeline file for that to work, but it shouldn't be too complicated to achieve if necessary. </div><div dir="auto"><br></div><div dir="auto">Note that the new build directory layout should be forced to on for Windows, as tests will never pass there due to how Qt on Windows works with QStandardPaths. </div><div dir="auto"><br></div><div dir="auto"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Perhaps in the long term also the logic in ECM needs to be changed, so the<br>
new build artifact placement can be triggered by other means from cmake code<br>
which is prepared for the optional new feature (by testing the version of the<br>
found ECM or some set feature flag variable). It will be quite some time for<br>
many application projects to jump to 5.38.0 as minimum ECM, given developers<br>
are sometimes bound to LTS distributions.<br>
<br>
Cheers<br>
<font color="#888888">Friedrich</font></blockquote></div></div></div><div dir="auto"><br></div><div dir="auto">Cheers,</div><div dir="auto">Ben</div><div dir="auto"></div></div>