Re: Review Request 119524: Force OpenGLES if on Qt 5.4

2014-08-23 Thread Alexander Richardson
On Aug. 11, 2014, 8:35 vorm., Martin Gräßlin wrote: I highly suggest to revert this change. Enforcing GLES means limiting to GLES (it's a subset after all) and also not all drivers do support GLES - that's still the smaller part. We are calling for lots of trouble if we go this

Re: Review Request 119524: Force OpenGLES if on Qt 5.4

2014-08-23 Thread David Edmundson
On Aug. 11, 2014, 6:35 a.m., Martin Gräßlin wrote: I highly suggest to revert this change. Enforcing GLES means limiting to GLES (it's a subset after all) and also not all drivers do support GLES - that's still the smaller part. We are calling for lots of trouble if we go this route.

Re: Review Request 119524: Force OpenGLES if on Qt 5.4

2014-08-21 Thread Marco Martin
hmm, i see it's still trying to set opengles, that shuld be reverted as was determined wasn't a good idea On Monday 11 August 2014 09:03:23 Martin Gräßlin wrote: On Aug. 11, 2014, 8:35 a.m., Martin Gräßlin wrote: I highly suggest to revert this change. Enforcing GLES means limiting to

Re: Review Request 119524: Force OpenGLES if on Qt 5.4

2014-08-11 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119524/#review64219 --- I highly suggest to revert this change. Enforcing GLES means

Re: Review Request 119524: Force OpenGLES if on Qt 5.4

2014-08-11 Thread Aleix Pol Gonzalez
On Aug. 11, 2014, 6:35 a.m., Martin Gräßlin wrote: I highly suggest to revert this change. Enforcing GLES means limiting to GLES (it's a subset after all) and also not all drivers do support GLES - that's still the smaller part. We are calling for lots of trouble if we go this route.

Re: Review Request 119524: Force OpenGLES if on Qt 5.4

2014-08-11 Thread Martin Gräßlin
On Aug. 11, 2014, 8:35 a.m., Martin Gräßlin wrote: I highly suggest to revert this change. Enforcing GLES means limiting to GLES (it's a subset after all) and also not all drivers do support GLES - that's still the smaller part. We are calling for lots of trouble if we go this route.

Re: Review Request 119524: Force OpenGLES if on Qt 5.4

2014-07-29 Thread Mark Gaiser
On jul 28, 2014, 7:03 p.m., Mark Gaiser wrote: If there is one hardware combination that works best on linux, it's OpenGL + Intel. I find it hard to believe that GLES is better and in all cases. That must be a driver/mesa bug that should be investigated and fixed. If this

Re: Review Request 119524: Force OpenGLES if on Qt 5.4

2014-07-29 Thread Eike Hein
On July 28, 2014, 6:25 p.m., Eike Hein wrote: Can you recompile plasma-desktop with OpenGL ES and test if the drag pixmap for Folder icons works? I wrote the code in a way that ought to work on both desktop GL and ES, but it's worth testing. Aleix Pol Gonzalez wrote: I have

Re: Review Request 119524: Force OpenGLES if on Qt 5.4

2014-07-29 Thread Sebastian Kügler
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119524/#review63437 --- Ship it! I'm all for this, as it means we reduce the set of

Re: Review Request 119524: Force OpenGLES if on Qt 5.4

2014-07-29 Thread David Edmundson
On July 29, 2014, 2:01 p.m., Sebastian Kügler wrote: I'm all for this, as it means we reduce the set of features we use from the underlying stack. We're even focusing on the features that will get more testing (since it's mobile, and that's the hype). I consider it an advantage if we

Re: Review Request 119524: Force OpenGLES if on Qt 5.4

2014-07-29 Thread Sebastian Kügler
On July 29, 2014, 2:01 p.m., Sebastian Kügler wrote: I'm all for this, as it means we reduce the set of features we use from the underlying stack. We're even focusing on the features that will get more testing (since it's mobile, and that's the hype). I consider it an advantage if we

Re: Review Request 119524: Force OpenGLES if on Qt 5.4

2014-07-29 Thread Aleix Pol Gonzalez
On July 29, 2014, 2:01 p.m., Sebastian Kügler wrote: I'm all for this, as it means we reduce the set of features we use from the underlying stack. We're even focusing on the features that will get more testing (since it's mobile, and that's the hype). I consider it an advantage if we

Re: Review Request 119524: Force OpenGLES if on Qt 5.4

2014-07-29 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119524/ --- (Updated July 29, 2014, 4:15 p.m.) Status -- This change has been

Review Request 119524: Force OpenGLES if on Qt 5.4

2014-07-28 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119524/ --- Review request for Plasma. Repository: plasma-workspace Description

Re: Review Request 119524: Force OpenGLES if on Qt 5.4

2014-07-28 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119524/#review63360 --- I still don't have 5.4 to test for now.. but, are you sure

Re: Review Request 119524: Force OpenGLES if on Qt 5.4

2014-07-28 Thread Mark Gaiser
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119524/#review63366 --- If there is one hardware combination that works best on

Re: Review Request 119524: Force OpenGLES if on Qt 5.4

2014-07-28 Thread Aleix Pol Gonzalez
On July 28, 2014, 6:25 p.m., Eike Hein wrote: Can you recompile plasma-desktop with OpenGL ES and test if the drag pixmap for Folder icons works? I wrote the code in a way that ought to work on both desktop GL and ES, but it's worth testing. I have it running with OpenGLES at the

Re: Review Request 119524: Force OpenGLES if on Qt 5.4

2014-07-28 Thread Aleix Pol Gonzalez
On July 28, 2014, 7:03 p.m., Mark Gaiser wrote: If there is one hardware combination that works best on linux, it's OpenGL + Intel. I find it hard to believe that GLES is better and in all cases. That must be a driver/mesa bug that should be investigated and fixed. If this