> On April 11, 2014, 5:32 a.m., Martin Gräßlin wrote: > > I'm wondering whether it's still needed in kwindowsystem at all. So the > > test might not be telling much. > > Martin Gräßlin wrote: > my task to remove the dependency from kwindowsystem failed: it's already > no longer used. > > I'm just wondering where we have code which still uses this X11_XCB, it > might be possible to delete it. It's in fact only needed for Qt4 as with Qt5 > one gets the xcb_connection_t* through QX11Info.
KIdleTime still uses it. LXR suggests it's the only user. I'd be for removing it if the beta version hadn't already been released. While not technically part of frameworks, I think ECM should still stick with the "SC after beta" thing. In this case, I'm not too bothered about leaving it in. It provides a sort of completeness with FindXCB and CMake's FindX11. - Alex ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117491/#review55425 ----------------------------------------------------------- On April 11, 2014, 8:32 p.m., Alex Merry wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/117491/ > ----------------------------------------------------------- > > (Updated April 11, 2014, 8:32 p.m.) > > > Review request for Build System, Extra Cmake Modules, KDE Frameworks, and > Martin Gräßlin. > > > Repository: extra-cmake-modules > > > Description > ------- > > Rework FindX11_XCB.cmake > > Imported target, version handling, package description etc. > > > Diffs > ----- > > find-modules/FindX11_XCB.cmake 687a4f3d57f67aa2e35a8bcfe201e0324e84204e > > Diff: https://git.reviewboard.kde.org/r/117491/diff/ > > > Testing > ------- > > Configured and build KIdleTime successfully. > > > Thanks, > > Alex Merry > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
