Review Request 116653: Factor out common code for component-based find modules
Alex Merry
alex.merry at kde.org
Fri Mar 7 20:04:16 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/116653/
-----------------------------------------------------------
Review request for Build System, Extra Cmake Modules and KDE Frameworks.
Repository: extra-cmake-modules
Description
-------
Factor out common code for component-based find modules
This comes with improvements for FindWayland and FindXCB:
- <module>_VERSION instead of <module>_VERSION_STRING
- set <module>_LIBRARIES to contain targets, so that
<module>_INCLUDE_DIRS and <module>_DEFINITIONS are unnecessary
- do pkg-config calls per-component
- handle inter-component dependencies
- improve documentation (in CMake 3.0 style)
- improve license texts (following CMake's style)
In FindWayland:
- extract the version from wayland-version.h if necessary
Diffs
-----
find-modules/FindWayland.cmake a121de9f8adda1e035b0f8e8401be26a45b61af4
find-modules/FindXCB.cmake eb3075885bc408fc4c213516a387dfcd0e75cb8b
modules/ECMFindModuleHelpers.cmake PRE-CREATION
Diff: https://git.reviewboard.kde.org/r/116653/diff/
Testing
-------
Tested various configurations with a test CMakeLists.txt file, including with various include files missing.
Configured and built kwindowsystem and kde-workspace.
Thanks,
Alex Merry
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140307/0477e64c/attachment-0001.html>
More information about the Kde-frameworks-devel
mailing list