Re: Issues with Qt5/mesa and eglfs

[Paul Eggleton <paul.eggleton@linux.intel.com>]

On Tuesday 25 June 2013 13:59:55 Nicolas Dechesne wrote:
> On Thu, Jun 20, 2013 at 8:15 PM, Nicolas Dechesne <
> nicolas.dechesne@linaro.org> wrote:
> > On Thu, Jun 20, 2013 at 11:46 AM, Nicolas Dechesne <
> > 
> > nicolas.dechesne@linaro.org> wrote:
> >> On Thu, Jun 20, 2013 at 11:42 AM, Paul Eggleton <
> >> 
> >> paul.eggleton@linux.intel.com> wrote:
> >>> I wasn't watching closely but my impression is there were a large number
> >>> of
> >>> changes associated with the mesa upgrade. Is the upgrade itself
> >>> necessary for
> >>> the fix?
> >> 
> >> no, i don't think so. I think we might be able to just update the patches
> >> that deal with the X11/no X11 support. but i haven't tested that yet. i
> >> just did a quick test by cherry picking the complete commit (e.g. mesa
> >> upgrade) to verify that it's actually fixing my problem. now that i know
> >> it
> >> does, i will try to just upgrade the patches without the mesa upgrade and
> >> report how it goes.
> > 
> > ok, i was able to backport the mesa patch from 9.1.3 to 9.0.2
> > (EGL-Mutate-NativeDisplayType-depending-on-config.patch) , the backport
> > was
> > more or less trivial. and then i did this in dylan branch:
> > 
> > --- a/meta/recipes-graphics/mesa/mesa-9.0.2.inc
> > +++ b/meta/recipes-graphics/mesa/mesa-9.0.2.inc
> > @@ -2,7 +2,7 @@ SRC_URI = "
> > ftp://ftp.freedesktop.org/pub/mesa/${PV}/MesaLib-${PV}.tar.bz2 \
> > 
> >             file://0002-cross-compile.patch \
> >             file://cross-glsl.patch \
> >             file://dont-fail-if-libX11-isnt-installed.patch \
> > 
> > -           file://fix-egl-compilation-without-x11-headers.patch \
> > +           file://EGL-Mutate-NativeDisplayType-depending-on-config.patch
> > \
> > 
> >             "
> > 
> > and it does fix the build issue i had (mesa/qt5/non X11 distro).
> > should that be merged into dylan (since it's merged in master already) or
> > do we need to wait for the 'final' patch in upstream to be ready?
> 
> Paul, Martin, what do you think about this? i have the (trivial) dylan
> backport ready. Do you think it's the right time to get it merged?

I'd say please send the patch to the mailing list (marked with [dylan]) and 
then we can review it. By the sounds of things it should be OK.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre