Review Request 111612: Add basic support for clang in FindKDE4Internal.cmake

Milian Wolff mail at milianw.de
Sat Jul 20 01:18:57 UTC 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111612/
-----------------------------------------------------------

Review request for Build System, Andreas Pakulat, Alexander Neundorf, and Stephen Kelly.


Description
-------

When building KDE applications with clang, multiple issues arise:

- debugfull is not supported at all, it's as if one gives an 'invalid' build type to cmake (i.e. neither -O nor -g is present)
- most of the "pedantic" flags are not set properly, such as those that ensure proper visibility macros etc. pp.
- none of the above is communicated to the developer, the might compile for him but not work as expected or he will introduce issues for others (like the windows guys)


Diffs
-----

  cmake/modules/FindKDE4Internal.cmake 98a6f48 

Diff: http://git.reviewboard.kde.org/r/111612/diff/


Testing
-------

build kate, kdevplatform and kdevelop in debugfull with clang 3.3 - finally have my debug symbols back!


Thanks,

Milian Wolff

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20130720/384b76ed/attachment.html>


More information about the Kde-buildsystem mailing list