Review Request 119564: Add define to re-enable Qt functionality we depend on.

Axel Rasmussen axel.rasmussen1 at gmail.com
Tue Aug 5 07:07:04 UTC 2014


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

(Updated Aug. 5, 2014, 7:07 a.m.)


Status
------

This change has been discarded.


Review request for Build System, Extra Cmake Modules and KDE Frameworks.


Bugs: 337472
    http://bugs.kde.org/show_bug.cgi?id=337472


Repository: extra-cmake-modules


Description
-------

Upstream Qt commit e112c2e altered the way QExplicitlySharedDataPointer behaves by default, such that it no longer uses a `static_cast` to cast compatible pointers. A nontrivial amount of KDE code (I encountered the build error in KService, although there are other users) depends on this functionality, so this commit adds a define to the build system which re-enables the code in Qt.


Diffs
-----

  kde-modules/KDECompilerSettings.cmake fdc930e 

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


Testing
-------

I applied this patch, attempted to build KService, and the build succeeded, rather than exhibiting the compile errors found in the log provided on the bug report.


Thanks,

Axel Rasmussen

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


More information about the Kde-buildsystem mailing list