Summary: | [kde overlay] dev-util/kdevplatform-9999 fails to build - Target "test_highlighting" links to target "Qt5::Test" but the target was not found. | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | CT <zarathustra.zoroaster> |
Component: | Current packages | Assignee: | Gentoo KDE team <kde> |
Status: | RESOLVED FIXED | ||
Severity: | normal | ||
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- |
Description
CT
2017-08-07 13:35:36 UTC
This is definitely a downstream issue, as our patching to make tests optional is all custom. Looking at sample failing target in language/highlighting/tests/CMakeLists.txt: ecm_add_test(test_highlighting.cpp LINK_LIBRARIES KF5::TextEditor Qt5::Test KDev::Tests KDev::Language) I suspect something elsewhere has changed, as IIRC ecm_add_test is a noop unless testing is enabled. (In reply to Michael Palimaka (kensington) from comment #1) > This is definitely a downstream issue, as our patching to make tests > optional is all custom. > > Looking at sample failing target in > language/highlighting/tests/CMakeLists.txt: > ecm_add_test(test_highlighting.cpp > LINK_LIBRARIES KF5::TextEditor Qt5::Test KDev::Tests KDev::Language) > > I suspect something elsewhere has changed, as IIRC ecm_add_test is a noop > unless testing is enabled. It seems the expected noop is not actually its current behavior, re: this patch https://phabricator.kde.org/D7187. https://phabricator.kde.org/D7187 has been pushed. You might want to use that patch in the extra-cmake-modules recipe. There have been a couple of commits to kdevplatform 5.1 branch, too, which allow building it without tests enabled. Just pass -DBUILD_TESTING=OFF to cmake. Thanks! Commit: 632e3f1055402b7b1ba939e22bbe884f159262b5 From: Andreas Sturmlechner <asturm@gentoo.org> Date: Mon, 7 Aug 2017 21:19:26 +0200 Subject: [PATCH] dev-util/kdevplatform: Switch KDE_TEST back to "true" Upstream commit 274260d52e127f92551742e969e12c478c7d7dae --- dev-util/kdevplatform/kdevplatform-5.1.9999.ebuild | 2 +- dev-util/kdevplatform/kdevplatform-9999.ebuild | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) (In reply to Kevin Funk from comment #3) > https://phabricator.kde.org/D7187 has been pushed. You might want to use > that patch in the extra-cmake-modules recipe. It was backported to 5.37, thanks. |