[PATCH] qt4: disable qmeegographicssystemhelper

[PATCH] qt4: disable qmeegographicssystemhelper

[Paul Eggleton]

We don't want this enabled just because we have EGL and OpenGL ES 2.0
enabled.

Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
---
 meta/recipes-qt/qt4/qt4-4.8.4.inc                  |  1 +
 ...ls.pro-disable-qmeegographicssystemhelper.patch | 28 ++++++++++++++++++++++
 2 files changed, 29 insertions(+)
 create mode 100644 meta/recipes-qt/qt4/qt4-4.8.4/0027-tools.pro-disable-qmeegographicssystemhelper.patch

diff --git a/meta/recipes-qt/qt4/qt4-4.8.4.inc b/meta/recipes-qt/qt4/qt4-4.8.4.inc
index e87dacc..0a8e00b 100644
--- a/meta/recipes-qt/qt4/qt4-4.8.4.inc
+++ b/meta/recipes-qt/qt4/qt4-4.8.4.inc
@@ -26,6 +26,7 @@ SRC_URI = "http://releases.qt-project.org/qt4/source/qt-everywhere-opensource-sr
            file://0024-Change-all-shmget-calls-to-user-only-memory.patch \
            file://0025-Remove-unnecessary-typedef-found-by-gcc4-8.patch \
            file://0026-Don-t-overwrite-QMAKE_QT_CONFIG-with-empty-value.patch \
+           file://0027-tools.pro-disable-qmeegographicssystemhelper.patch \
            file://g++.conf \
            file://linux.conf \
            "
diff --git a/meta/recipes-qt/qt4/qt4-4.8.4/0027-tools.pro-disable-qmeegographicssystemhelper.patch b/meta/recipes-qt/qt4/qt4-4.8.4/0027-tools.pro-disable-qmeegographicssystemhelper.patch
new file mode 100644
index 0000000..6e30aee
--- /dev/null
+++ b/meta/recipes-qt/qt4/qt4-4.8.4/0027-tools.pro-disable-qmeegographicssystemhelper.patch
@@ -0,0 +1,28 @@
+From 0460e2a5f13aaf0081c3c9fec89280046842ddbb Mon Sep 17 00:00:00 2001
+From: Paul Eggleton <paul.eggleton@linux.intel.com>
+Date: Wed, 3 Jul 2013 17:48:30 +0100
+Subject: [PATCH] tools.pro: disable qmeegographicssystemhelper
+
+We don't want this enabled just because we have EGL and OpenGL ES 2.0
+enabled.
+
+Upstream-Status: Inappropriate [config]
+
+Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
+---
+ tools/tools.pro | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/tools/tools.pro b/tools/tools.pro
+index 0e27053..2d3f36d 100644
+--- a/tools/tools.pro
++++ b/tools/tools.pro
+@@ -49,4 +49,4 @@ QTDIR_build:REQUIRES = "contains(QT_CONFIG, full-config)"
+ 
+ !win32:!embedded:!mac:!symbian:CONFIG += x11
+ 
+-x11:contains(QT_CONFIG, opengles2):contains(QT_CONFIG, egl):SUBDIRS += qmeegographicssystemhelper
++#x11:contains(QT_CONFIG, opengles2):contains(QT_CONFIG, egl):SUBDIRS += qmeegographicssystemhelper
+-- 
+1.8.1.2
+
-- 
1.8.1.2

Re: [PATCH] qt4: disable qmeegographicssystemhelper

[Otavio Salvador]

On Wed, Jul 3, 2013 at 2:04 PM, Paul Eggleton
<paul.eggleton@linux.intel.com> wrote:
> We don't want this enabled just because we have EGL and OpenGL ES 2.0
> enabled.
>
> Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>

Agreed; maybe this could be considered for dylan as well?

--
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750

Re: [PATCH] qt4: disable qmeegographicssystemhelper

[Paul Eggleton]

On Thursday 04 July 2013 10:10:52 Otavio Salvador wrote:
> On Wed, Jul 3, 2013 at 2:04 PM, Paul Eggleton
> 
> <paul.eggleton@linux.intel.com> wrote:
> > We don't want this enabled just because we have EGL and OpenGL ES 2.0
> > enabled.
> > 
> > Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
> 
> Agreed; maybe this could be considered for dylan as well?

Sounds reasonable to me; once it's merged to master I will include it in my 
queue for dylan. 

To be honest it would be nice if it were fixed properly upstream; building it 
based on the conditional given seems wrong.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre

Re: [PATCH] qt4: disable qmeegographicssystemhelper

[Otavio Salvador]

On Thu, Jul 4, 2013 at 10:20 AM, Paul Eggleton
<paul.eggleton@linux.intel.com> wrote:
> On Thursday 04 July 2013 10:10:52 Otavio Salvador wrote:
>> On Wed, Jul 3, 2013 at 2:04 PM, Paul Eggleton
>>
>> <paul.eggleton@linux.intel.com> wrote:
>> > We don't want this enabled just because we have EGL and OpenGL ES 2.0
>> > enabled.
>> >
>> > Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
>>
>> Agreed; maybe this could be considered for dylan as well?
>
> Sounds reasonable to me; once it's merged to master I will include it in my
> queue for dylan.
>
> To be honest it would be nice if it were fixed properly upstream; building it
> based on the conditional given seems wrong.

I fully agree but until it is fixed (if it is ever fixed) your patch
seems the way to go! :-)

--
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750