Review Request: Disable X11 support in plasma WindowShadows on Qt5, as the QPixmap X11 integration is gone.

Tue Oct 23 12:53:11 UTC 2012

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

(Updated Oct. 23, 2012, 12:53 p.m.)

Review request for KDE Frameworks.

Changes
-------

New version, #pragma message() added.

Description
-------

This essentially makes WindowShadows a no-op, but it is better than not compiling...

The issue is currently hidden by Plasma not being built at all on Qt5, but this should be fixed before my "Drop KDE_NO_PHONON hackery from KDE Frameworks" patch are merged, or Qt5 compilation will break.

Diffs (updated)
-----

  plasma/private/windowshadows.cpp 645dddd 

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

Testing
-------

Thanks,

Jon Severinsson

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