Review Request 124318: Load DPMSAction only on xcb platform

[Kai Uwe Broulik]

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

Ship it!


Thanks!

- Kai Uwe Broulik


On Juli 10, 2015, 9:21 vorm., Martin Gräßlin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124318/
> -----------------------------------------------------------
> 
> (Updated Juli 10, 2015, 9:21 vorm.)
> 
> 
> Review request for Plasma and Solid.
> 
> 
> Repository: powerdevil
> 
> 
> Description
> -------
> 
> The DPMS action is implemented in a very x11 specific way calling into
> xcb in the ctor and before supported can be checked. This causes a crash
> on platforms != xcb.
> 
> In order to better support other platforms the action is marked as
> xcb specific in the desktop file and the loading of Actions is adjusted
> to only load Actions supported on the current platform.
> 
> 
> Diffs
> -----
> 
>   daemon/actions/dpms/powerdevildpmsaction.desktop 78c5d90fe62559f70431dd4bd6e195b765e0f477 
>   daemon/powerdevilactionpool.cpp 0f0e69f0afe447935e6d17872f029c7c4ac33875 
> 
> Diff: https://git.reviewboard.kde.org/r/124318/diff/
> 
> 
> Testing
> -------
> 
> No longer crashes in a Plasma/Wayland session.
> 
> 
> Thanks,
> 
> Martin Gräßlin
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150710/7e2af11d/attachment-0001.html>