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 > > thi

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 > > th

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
> 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 > > thi

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 > > thi

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

2014-08-10 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 l

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 ma

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 > > advanta

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 > > advanta

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 > > advanta

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 f

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:

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

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

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 OpenGL

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 linux,

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

2014-07-28 Thread Eike Hein
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119524/#review63362 --- Can you recompile plasma-desktop with OpenGL ES and test if th

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 thi

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 ---