From: Marc Date: Thu, 3 Sep 2015 13:22:10 +0000 (+0200) Subject: qt5base: Fix compile errors in qurl.h / qeglplatformscreen.cpp X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=b2a4fb24591c398cbf3a7c8cbd8e05910eb681db;p=buildroot.git qt5base: Fix compile errors in qurl.h / qeglplatformscreen.cpp Update the already existing fix for EGL/X11 header issue to fix an additional problem encountered on my system where I had compile errors in qeglplatformscreen.cpp. The problem was related to the wrong order of includes. The X11 headers must always be included last, as indicated in http://lists.qt-project.org/pipermail/development/2013-March/010511.html The fix is done in the existing 0003-xcb-egl-fixes.patch patch, since it is an additional fix for the same problem. [Thomas: tweak commit log, and adjust SoB details as suggested by Arnout.] Signed-off-by: Marc Andre Reviewed-by: Arnout Vandecappelle (Essensium/Mind) Signed-off-by: Thomas Petazzoni --- diff --git a/package/qt5/qt5base/0003-xcb-egl-fixes.patch b/package/qt5/qt5base/0003-xcb-egl-fixes.patch index 53f8453461..65d6f6921c 100644 --- a/package/qt5/qt5base/0003-xcb-egl-fixes.patch +++ b/package/qt5/qt5base/0003-xcb-egl-fixes.patch @@ -1,12 +1,15 @@ -The vivante headers pull in X11 defines which break eglconvenience +Some EGL headers pull in X11 defines which break eglconvenience * Adds a missing include to qeglplatformcontext.cpp * Fix namespace collision on CursorShape, pulled in from X11/X.h * Do not pass MESA_EGL_NO_X11_HEADERS when xcb is use +* Reorder includes to have the X11-header the last include [based on patch from Yocto project] Signed-off-by: Jérôme Pouiller - +Updated to fix issue in qeglplatformscreen (reorder includes, see also +http://lists.qt-project.org/pipermail/development/2013-March/010511.html) +Signed-off-by: Marc Andre diff -Nuar a/src/platformsupport/eglconvenience/eglconvenience.pri b/src/platformsupport/eglconvenience/eglconvenience.pri --- a/src/platformsupport/eglconvenience/eglconvenience.pri 2014-08-07 16:53:50.038277168 -0400 @@ -61,4 +64,19 @@ diff -Nuar a/src/platformsupport/eglconvenience/qeglplatformcontext.cpp b/src/pl #include "qeglplatformscreen_p.h" QT_BEGIN_NAMESPACE +--- qtbase-opensource-src-5.5.0.orig/src/platformsupport/eglconvenience/qeglplatformscreen.cpp 2015-09-01 14:46:57.116883353 +0200 ++++ qtbase-opensource-src-5.5.0/src/platformsupport/eglconvenience/qeglplatformscreen.cpp 2015-09-01 14:47:36.560882333 +0200 +@@ -31,11 +31,11 @@ + ** + ****************************************************************************/ + +-#include "qeglplatformscreen_p.h" +-#include "qeglplatformwindow_p.h" + #include + #include + #include ++#include "qeglplatformscreen_p.h" ++#include "qeglplatformwindow_p.h" + + QT_BEGIN_NAMESPACE