Review Request 112901: Remove weird cmake indirections
Aleix Pol Gonzalez
aleixpol at kde.org
Tue Sep 24 11:36:12 UTC 2013
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112901/
-----------------------------------------------------------
(Updated Sept. 24, 2013, 11:36 a.m.)
Review request for Build System and KDE Frameworks.
Changes
-------
Don't specifically say we want SHARED libraries, let cmake use its defaults wisely.
Description
-------
There were some un-reviewed cmake files where libraries would be defined with a ${LIBRARY_TYPE}. It seems to be something coming from the KDE4 times that didn't follow through.
I propose to remove all the uses for the moment, it's not ever being set anyway.
Diffs (updated)
-----
kdesu/CMakeLists.txt e526643
interfaces/kmediaplayer/CMakeLists.txt 3d1797d
interfaces/kimproxy/library/CMakeLists.txt 36e55ef
kdewebkit/CMakeLists.txt cdc5835
kfile/CMakeLists.txt e05137d
khtml/CMakeLists.txt 4e1bb80
kio/CMakeLists.txt 25dc6d9
kio/misc/kntlm/CMakeLists.txt fe70a8d
kjsembed/kjsembed/CMakeLists.txt 8ed55c1
knewstuff/src/CMakeLists.txt c313981
knotify/config/CMakeLists.txt 69e1c11
kparts/CMakeLists.txt 6ab9391
kpty/CMakeLists.txt 9a79827
kross/core/CMakeLists.txt 52cd3d4
kross/qts/CMakeLists.txt 1a30bd1
kross/ui/CMakeLists.txt 9e7806d
kutils/CMakeLists.txt 0cb281d
staging/kde4support/CMakeLists.txt 24ee457
staging/kemoticons/src/core/CMakeLists.txt f7fb463
staging/ktextwidgets/src/CMakeLists.txt 4787c10
Diff: http://git.reviewboard.kde.org/r/112901/diff/
Testing
-------
Everything still builds
Thanks,
Aleix Pol Gonzalez
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20130924/d4fbdba1/attachment-0001.html>
More information about the Kde-buildsystem
mailing list