Review Request 124695: Drop build option KWIN_BUILD_EGL

Martin Gräßlin mgraesslin at kde.org
Tue Aug 11 09:12:40 UTC 2015



> On Aug. 11, 2015, 11:10 a.m., Thomas Lübking wrote:
> > Two notes:
> > a) no idea how epoxy works, but it doesn't link any GL library itself here (it's an epoxy build time dependency, though)
> > b) fglrx doesn't support EGL - though rather irrelevant since EGL from MESA will just be installed (as qt5-base dependency already) and not used, afaics there actually is a way to build a GLX system only.

concerning a: yes that's just a miswording from my side. What I wanted to express is that we don't link it explicitly any more since the switch to epoxy

concerning b: is that still the case? I would have assumed that with the new driver architecture they also got EGL working?


- Martin


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


On Aug. 11, 2015, 10:29 a.m., Martin Gräßlin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124695/
> -----------------------------------------------------------
> 
> (Updated Aug. 11, 2015, 10:29 a.m.)
> 
> 
> Review request for kwin and Plasma.
> 
> 
> Repository: kwin
> 
> 
> Description
> -------
> 
> It doesn't make much sense any more as we get EGL linked and included
> through epoxy anyway. Even more on Wayland it's just plain stupid to
> have EGL disabled. So removing the option just simplifies our code base
> without any disadvantages.
> 
> 
> Diffs
> -----
> 
>   libkwineffects/kwinglutils.cpp 66634d6795d43020e9f323af9c24d8bfae9bf4aa 
>   libkwineffects/kwinconfig.h.cmake 3e34723aea5baf519c16f53afef3b21ab43efe7c 
>   dbusinterface.cpp 01079a1b2a97c83e4624deafb288349ea3952c7f 
>   backends/x11/x11windowed_backend.cpp 3ba9e5cd3932574daff2062a289dfd00745fa776 
>   backends/wayland/CMakeLists.txt a1670a34b1f6519f6fb37bc041516bf3aec508e1 
>   CMakeLists.txt 4d2aeed4f5ee9d0765e1b902ac51859e0ae1150b 
>   libkwineffects/kwinglutils_funcs.h eea28c5ae4225939a3d7fe72167fea4bfe028298 
>   libkwineffects/kwinglutils_funcs.cpp 4e763b7bb1c4e156b52639dd357c92c5f28e9937 
>   options.cpp 39efd9de53024fb7f11d042b7877a4583350b651 
>   scene_opengl.cpp 85c0b104785ee2f06b1698d8e8020e0615fdab88 
> 
> Diff: https://git.reviewboard.kde.org/r/124695/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Martin Gräßlin
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150811/dedda579/attachment.html>


More information about the Plasma-devel mailing list