Review Request 125236: FindQt4: Use CHECK_CXX_SYMBOL_EXISTS instead of CHECK_C_SYMBOL_EXISTS.

Raphael Kubo da Costa rakuco at FreeBSD.org
Sun Nov 29 18:15:44 UTC 2015


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

(Updated Nov. 29, 2015, 8:15 p.m.)


Status
------

This change has been discarded.


Review request for Build System and kdelibs.


Repository: kdelibs


Description
-------

Backport upstream CMake commit de30173d ("Remove C compiler requirement
from FindQt4.cmake") from 2011 so that we check for symbols such as the
Q_WS_* ones with a C++ compiler instead of a C one.


Diffs
-----

  cmake/modules/FindQt4.cmake e439a72f82c51758830b057b368407e11a6a541e 

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


Testing
-------

Adding something C++-specific to qglobal.h (e.g. `#include <ciso646>`)
works and the right flags are still detected.


Thanks,

Raphael Kubo da Costa

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20151129/73e19182/attachment.html>


More information about the Kde-buildsystem mailing list