Review Request 119588: Remove Qt5 debugfull hack from ECMConfig.cmake

[Alex Merry]

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

(Updated Aug. 6, 2014, 8:22 p.m.)


Status
------

This change has been marked as submitted.


Review request for Extra Cmake Modules and KDE Frameworks.


Repository: extra-cmake-modules


Description
-------

The original purpose of this was to set the QT_NO_DEBUG macro if the old
DebugFull configuration was used. We got rid of DebugFull (instead just
using Debug), so it is no longer relevant, and this hack never belonged
in ECMConfig.cmake anyway (it should have been in KDECompilerSettings).

CHANGELOG: ECM now works when the required CMake version is set to
3.0.0 or higher, and does not require Qt5Core to be available.

BUG: 331849


Diffs
-----

  ECMConfig.cmake.in ea47abf0d8b025dd9e424b0aea26986d1f288279 

Diff: https://git.reviewboard.kde.org/r/119588/diff/


Testing
-------

kimageformats configures when the required CMake version is set to 3.0.0 in (before, the configure step would fail).

oxygen-fonts configures when /usr/lib/cmake/Qt5Core is removed (before, the configure step would fail).


Thanks,

Alex Merry

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140806/e89a5568/attachment-0001.html>