Review Request: Don't lose the original CMAKE_MODULE_PATH
Patrick Spendrin
ps_ml at gmx.de
Sun Aug 5 09:32:06 UTC 2012
> On Aug. 5, 2012, 9:12 a.m., David Faure wrote:
> > tier2/kconfig/CMakeLists.txt, line 7
> > <http://git.reviewboard.kde.org/r/105863/diff/1/?file=76038#file76038line7>
> >
> > Ah, damn, there are conflicting goals here.
> >
> > We removed ${CMAKE_MODULE_PATH} on purpose so that each framework (in this case, tier2/kconfig) builds standalone, i.e. doesn't use any cmake file from the rest of kdelibs. As long as we have independent frameworks being built together in one big kdelibs (which is a temporary situation), this is a way to ensure that each module is self-contained in terms of cmake files.
> >
> > Now let's talk about FindKDEWin.cmake: can't we get rid of that? Try to think of kconfig as "a pure Qt-based library", like say, soprano, qca, or qjson. These libs don't use and don't need FindKDEWin.cmake, right? So why would kconfig need that additional layer? I know it was a necessary layer to get KDE4 code to compile on Windows, but the goal with KF5 is to remove layers and ensure that Qt and cmake have everything we need to compile our standalone libs on top of them.
> >
> > Please evaluate what needs the "kdewin" (library, right?), and whether that code can't be ported to "pure Qt" instead.
Yes, when I started with kf5 I already tried to put the easy parts into frameworks itself. Some stuff will be really hard, especially where "excessive" use of posix functions is used. Iirc kconfig was one of these, so it might be needed/wanted to keep the dependency for that part. One issue I wanted to address is that kdewin should ship a cmake Config.cmake so that we at least do not need the FindKDEWin.cmake anymore.
The question for KDE actually is how much code from kdewin can/should go into those libraries before it is better to keep kdewin.
- Patrick
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/105863/#review16879
-----------------------------------------------------------
On Aug. 4, 2012, 9:11 p.m., Andrius da Costa Ribas wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105863/
> -----------------------------------------------------------
>
> (Updated Aug. 4, 2012, 9:11 p.m.)
>
>
> Review request for KDE Frameworks and Patrick Spendrin.
>
>
> Description
> -------
>
> Keep the paths already in CMAKE_MODULE_PATH when adding ECM_MODULE_PATH and other search-paths into it.
>
>
> Diffs
> -----
>
> CMakeLists.txt f20069c
> tier2/kconfig/CMakeLists.txt c4b2cf6
>
> Diff: http://git.reviewboard.kde.org/r/105863/diff/
>
>
> Testing
> -------
>
> Before this adjustment it was not possible to proceed with the build due to missing Find*.cmake (e.g.: FindKDEWin.cmake) files.
>
>
> Thanks,
>
> Andrius da Costa Ribas
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20120805/4539d845/attachment-0001.html>
More information about the Kde-frameworks-devel
mailing list