D10762: Fix the build on armhf/aarch64

[Martin Flöser]

graesslin requested changes to this revision.
graesslin added inline comments.
This revision now requires changes to proceed.
Restricted Application edited projects, added KWin; removed Plasma.

INLINE COMMENTS

> kwinglplatform.cpp:22
>  #include "kwinglplatform.h"
> +#include "kwinglutils_funcs.h"
>  #include <epoxy/gl.h>

why include kwinglutils_funcs.h? If it's truly needed, please add a comment so that it doesn't get removed with cleanup

> platform.h:24-37
>  #include <epoxy/egl.h>
> +
> +// qopengl.h declares GLdouble as a typedef of float when Qt is built
> +// with GLES support.  This conflicts with the epoxy/gl_generated.h
> +// declaration, so we have to prevent the Qt header from being #included.
> +#define QOPENGL_H
> +

Instead of copy pasting this all over the place: let's add one header file which does the epoxy include and the ifdef mess.

REPOSITORY
  R108 KWin

REVISION DETAIL
  https://phabricator.kde.org/D10762

To: bshah, #plasma, graesslin
Cc: plasma-devel, kwin, #kwin, iodelay, bwowk, ZrenBot, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180223/22ba6860/attachment.html>