Review Request 122320: use xcb-screen count instead of qguiapplication.screens

Nick Shaforostoff shafff at ukr.net
Mon Feb 2 21:15:04 GMT 2015


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

(Updated Feb. 2, 2015, 9:15 p.m.)


Review request for kde-workspace, Martin Gräßlin and Thomas Lübking.


Changes
-------

-declare XCB an optional dependency
-add a check of current qt platform (xcb/wayland)


Repository: plasma-workspace


Description
-------

this patch makes kcminit behave like in kde4: it uses proper xcb screen count which may be different from QGuiApplication::screens().count().

for example when i connext external monitor via vga to my laptop, xcb screen count is still '1', while QGuiApplication::screens().count() returns '2'.

switching from QGuiApplication to QCoreApplication still wasn't possible because modules like 'mouse' need gui initialized and would crash if kcminit uses QCoreApplication.


Diffs (updated)
-----

  components/CMakeLists.txt 42c820f 
  startkde/kcminit/CMakeLists.txt b17951f 
  startkde/kcminit/main.cpp 1008966 

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


Testing
-------

i have built kcminit on ubuntu vivid alpha 32-bit, replaced binaries and libraries in the system and successfuly could run kcminit_startup and reboot also went fine.


Thanks,

Nick Shaforostoff

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20150202/1def8956/attachment.htm>


More information about the kde-core-devel mailing list