Fix for CI needed: Running tests before install
Friedrich W. H. Kossebau
kossebau at kde.org
Wed Aug 30 04:12:04 UTC 2017
Am Montag, 21. August 2017, 12:10:15 CEST schrieb Kevin Funk:
> Heya,
>
> with David Faure's latest change in ECM, it's now theoretically possible to
> run tests even without installing the project:
>
> https://blogs.kde.org/2017/08/15/running-applications-and-unittests-without
> -make-install
>
> The CI adapted to this, and now runs the test step before the install step.
Though chance is CI adapted not properly yet. And changed the order of running
tests and installing also for products where it should not have yet:
Reason is that code in ECM to support running tests uninstalled right now is
only triggered by some "find_package(ECM x.y.z)", with x.y.z >= 5.38.0
Cmp. the
NOT ("${ECM_GLOBAL_FIND_VERSION}" VERSION_LESS "5.38.0")
in https://cgit.kde.org/extra-cmake-modules.git/tree/kde-modules/
KDECMakeSettings.cmake#n252 , where ${ECM_GLOBAL_FIND_VERSION} is from
set(ECM_GLOBAL_FIND_VERSION "${ECM_FIND_VERSION}")
in https://cgit.kde.org/extra-cmake-modules.git/tree/ECMConfig.cmake.in#n11
And that find_package of ECM with 5.38.0 is something only the KDE Frameworks
repos do right now. Where other projects, like kdevelop, are fine with less
recent versions (kdevelop master only expects ECM 5.14.0 as minimum).
Chance is the hint for a scope on the requested build steps order change for
CI "For frameworks only, for now." in https://mail.kde.org/pipermail/kde-frameworks-devel/2017-August/048134.html was missed by CI maintainers.
I now replied to that thread pointing to this, let's see what their reply is.
Cheers
Friedrich
More information about the KDevelop-devel
mailing list