On Mon, 2015-07-27 at 11:00 -0700, Tobias Olausson wrote:
> On 27 July 2015 at 08:25, Martin Jansa <martin.ja...@gmail.com> 
> wrote:
> > On Tue, Jul 21, 2015 at 10:46:11AM -0700, Tobias Olausson wrote:
> > > In 10.4 versions of mesa, the check for MESA_EGL_NO_X11_HEADERS 
> > uses an #ifdef,
> > > not an #if define().
> > 
> > So this commit which was recently backported to fido branch should 
> > be
> > reverted:
> Well, reverting the already backported patch would break my patch. I 
> don't mind resubmitting a new patch, but the one I originally 
> submitted builds on top of the backported one, so the backported one 
> would be neutralized.

Ideally you'd submit a series of patches with 1) a patch to revert the
already backported patch below and 2) a patch to backport the change
from 9c746017af381884cc20c7cd563fc429c2c66112.

That would give us nice clean git history (making future patch
backporting easier) and mean next to no work for me.

Does that seem reasonable?

Regards,

Joshua

> >  
> > commit 12e467f436fbc22f274558c753f0ac9756ce1071
> > Author:     Robert Yang <liezhi.y...@windriver.com>
> > AuthorDate: Wed May 6 00:08:35 2015 -0700
> > Commit:     Richard Purdie <richard.pur...@linuxfoundation.org>
> > CommitDate: Sun Jun 28 09:41:54 2015 +0100
> > 
> >     mesa: fix do_install_append
> > 
> >     "ifdef MESA_EGL_NO_X11_HEADERS" -> "if 
> > defined(MESA_EGL_NO_X11_HEADERS)"
> > 
> >     (From OE-Core master rev: 
> > 3a464d67b60f70b865f7db768e7edc53e40ff450)
> > 
> >     Signed-off-by: Robert Yang <liezhi.y...@windriver.com>
> >     Signed-off-by: Richard Purdie <
> > richard.pur...@linuxfoundation.org>
> >     Signed-off-by: Joshua Lock <joshua.l...@collabora.co.uk>
> > 
> > with proper commit message (preferably separately from
> > 9c746017af381884cc20c7cd563fc429c2c66112 backport), otherwise I 
> > like
> > what this patch is doing.
> > 
> > Thanks
> > 
> > >
> > > Also, this backports commit 
> > 9c746017af381884cc20c7cd563fc429c2c66112
> > > to fido from master.
> > >
> > > Signed-off-by: Tobias Olausson <tobias.olaus...@pelagicore.com>
> > > ---
> > >  meta/recipes-graphics/mesa/mesa_10.4.4.bb | 2 +-
> > >  meta/recipes-graphics/mesa/mesa_git.bb    | 2 +-
> > >  2 files changed, 2 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/meta/recipes-graphics/mesa/mesa_10.4.4.bb 
> > b/meta/recipes-graphics/mesa/mesa_10.4.4.bb
> > > index 90cccbe..a5f510d 100644
> > > --- a/meta/recipes-graphics/mesa/mesa_10.4.4.bb
> > > +++ b/meta/recipes-graphics/mesa/mesa_10.4.4.bb
> > > @@ -11,6 +11,6 @@ S = "${WORKDIR}/Mesa-${PV}"
> > >  #make eglplatform.h independent of MESA_EGL_NO_X11_HEADER
> > >  do_install_append() {
> > >      if ${@bb.utils.contains('PACKAGECONFIG', 'egl', 'true', 
> > 'false', d)}; then
> > > -        sed -i -e 's/^#if defined(MESA_EGL_NO_X11_HEADERS)/#if 
> > ${@bb.utils.contains('PACKAGECONFIG', 'x11', '0', '1', d)}/' 
> > ${D}${includedir}/EGL/eglplatform.h
> > > +        sed -i -e 's/^#ifdef MESA_EGL_NO_X11_HEADERS/#if 
> > defined(MESA_EGL_NO_X11_HEADERS) || ${@bb.utils.contains('PACKAGECO
> > NFIG', 'x11', '0', '1', d)}/' ${D}${includedir}/EGL/eglplatform.h
> > >      fi
> > >  }
> > > diff --git a/meta/recipes-graphics/mesa/mesa_git.bb 
> > b/meta/recipes-graphics/mesa/mesa_git.bb
> > > index 1598019..269ae1e 100644
> > > --- a/meta/recipes-graphics/mesa/mesa_git.bb
> > > +++ b/meta/recipes-graphics/mesa/mesa_git.bb
> > > @@ -13,6 +13,6 @@ S = "${WORKDIR}/git"
> > >  #make eglplatform.h independent of MESA_EGL_NO_X11_HEADER
> > >  do_install_append() {
> > >      if ${@bb.utils.contains('PACKAGECONFIG', 'egl', 'true', 
> > 'false', d)}; then
> > > -        sed -i -e 's/^#if defined(MESA_EGL_NO_X11_HEADERS)/#if 
> > ${@bb.utils.contains('PACKAGECONFIG', 'x11', '0', '1', d)}/' 
> > ${D}${includedir}/EGL/eglplatform.h
> > > +        sed -i -e 's/^#ifdef MESA_EGL_NO_X11_HEADERS/#if 
> > defined(MESA_EGL_NO_X11_HEADERS) || ${@bb.utils.contains('PACKAGECO
> > NFIG', 'x11', '0', '1', d)}/' ${D}${includedir}/EGL/eglplatform.h
> > >      fi
> > >  }
> > > --
> > > 2.1.4
> > >
> > > --
> > > _______________________________________________
> > > Openembedded-core mailing list
> > > Openembedded-core@lists.openembedded.org
> > > http://lists.openembedded.org/mailman/listinfo/openembedded-core
> > 
> > --
> > Martin 'JaMa' Jansa     jabber: martin.ja...@gmail.com
> > 
> 
> 
> -- 
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
-- 
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Reply via email to