kde-frameworks/extra-cmake-modules: Allow ECMQueryQt lookups to be cached //and make use of it from ecm.eclass - #1101
Open
a17r wants to merge 18 commits into
Open
kde-frameworks/extra-cmake-modules: Allow ECMQueryQt lookups to be cached //and make use of it from ecm.eclass#1101a17r wants to merge 18 commits into
a17r wants to merge 18 commits into
Conversation
a17r
force-pushed
the
eapi9
branch
4 times, most recently
from
August 9, 2026 22:42
dcc60af to
e61d40c
Compare
| qtpaths unnecessary. | ||
|
|
||
| ECM may be used by small packages that otherwise do not (or just optionally) | ||
| need Qt for tooling nor build. |
Member
There was a problem hiding this comment.
Should we also specifically call out the fact that this makes cross compilation easier? At least for upstreaming the patch that may help make it an easier sell.
Member
Author
There was a problem hiding this comment.
I've mentioned it upstream, we'll see.
a17r
force-pushed
the
eapi9
branch
7 times, most recently
from
August 10, 2026 23:03
267f5ba to
8a5f1c6
Compare
thesamesam
approved these changes
Aug 12, 2026
| set(BUILD_WITH_QT6 ON CACHE BOOL "") # QtVersionOption.cmake: Hard-require Qt6 | ||
| # KDEInstallDirs6 section | ||
| set(KDE_INSTALL_USE_QT_SYS_PATHS ON CACHE BOOL "") # install mkspecs in same dir as Qt stuff | ||
| set(QT_HOST_LIBEXECS_CACHED ${BROOT}$(qt_get_libexecdir 6) CACHE PATH "") # FindQtWaylandScanner.cmake |
This was switched to ON back in 2014 (commit ce6dbc0) but has long been obsolete. If QT_INSTALL_PREFIX is the same as CMAKE_INSTALL_PREFIX then the default state assigned to KDE_INSTALL_USE_QT_SYS_PATHS by KDEInstallDirs6.cmake is ON anyway. Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>
ECMQueryQt.cmake module from kde-frameworks/extra-cmake-modules will be adapted to derive any respective QT_INSTALL_* from these vars. Bug: https://bugs.gentoo.org/939465 Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>
…ched Bug: https://bugs.gentoo.org/939465 Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>
ECM_AUTODEPS=gui is equivalent to ECM_NONGUI=false (default, ugly) ECM_AUTODEPS=base equiv. to ECM_NONGUI=true + BDEPEND on dev-qt/qtbase:6 ECM_AUTODEPS=minimal furthermore allows to drop dev-qt/qtbase:6 kde-frameworks/kf-env will be obsolete in >EAPI-8. Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>
No Qt dependency here. Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>
Adjustments have been made to ecm.eclass which make this possible - set ECM_AUTODEPS=minimal to avoid dev-qt/qtbase:6 dependency. Bug: https://bugs.gentoo.org/923502 Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>
No GUI application. Drop obsolete KFMIN setting below baseline. Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>
No GUI application. Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>
No Qt dependency. Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>
No GUI application. Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>
No GUI application. Fix HOMEPAGE. Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>
No Qt dependency. Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>
No Qt dependency. Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>
No Qt dependency. Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>
Set ECM_AUTODEPS=minimal - no Qt dependency. See also cc82dc4 Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>
Set ECM_AUTODEPS=base to avoid circular dependency on itself. Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>
No Qt dependency. Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>
No Qt dependency. Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
TODO: Decide what to do with BDEPEND on dev-qt/qtbase.