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:37 UTC 2015
> On Sept. 17, 2015, 1:01 p.m., Alex Merry wrote:
> > This change is a correct one to make, but I've slightly lost track of the kdelibs release policy.
> >
> > This patch isn't fixing an issue anyone is actually hitting, right? And it's not like qglobal.h in Qt 4 is going to have any significant changes made to it. So it's possible this doesn't pass the threshold of bug fixes for kdelibs, which is permanently frozen to some degree.
> This patch isn't fixing an issue anyone is actually hitting, right?
I was going to say we had to do it downstream for FreeBSD because the oldest supported release can end up using clang with a very old libstdc++ without C++11 support, but since then I've managed to put all the required changes in our Qt package :-)
- Raphael
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125236/#review85550
-----------------------------------------------------------
On Sept. 15, 2015, 11:59 a.m., Raphael Kubo da Costa wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125236/
> -----------------------------------------------------------
>
> (Updated Sept. 15, 2015, 11:59 a.m.)
>
>
> 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/c4a5ea0f/attachment.html>
More information about the Kde-buildsystem
mailing list