Re: [PATCH 1/1] libsdl: Add support for libsdl-native

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

On Tuesday 17 December 2013 18:51:08 Andrei Gherzan wrote:
> On Tue, Dec 17, 2013 at 6:47 PM, Paul Eggleton <
> 
> paul.eggleton@linux.intel.com> wrote:
> > On Tuesday 17 December 2013 18:33:51 Andrei Gherzan wrote:
> > > From: Alexandru Niculita <alexnick87@gmail.com>
> > > 
> > > Change-Id: I4fb1e623c7bbb1f35ae3001c839cd7e16ce1bd65
> > > Signed-off-by: Alexandru Niculita <alexnick87@gmail.com>
> > > ---
> > > 
> > >  meta/recipes-graphics/libsdl/libsdl_1.2.15.bb | 3 ++-
> > >  1 file changed, 2 insertions(+), 1 deletion(-)
> > > 
> > > diff --git a/meta/recipes-graphics/libsdl/libsdl_1.2.15.bb
> > > b/meta/recipes-graphics/libsdl/libsdl_1.2.15.bb index 4b4ff63..84d3ad6
> > > 100644
> > > --- a/meta/recipes-graphics/libsdl/libsdl_1.2.15.bb
> > > +++ b/meta/recipes-graphics/libsdl/libsdl_1.2.15.bb
> > > @@ -17,6 +17,7 @@ DEPENDS = "${@base_contains('DISTRO_FEATURES',
> > 
> > 'directfb',
> > 
> > > 'directfb', '', d)} \ ${@base_contains('DISTRO_FEATURES', 'x11',
> > > 'virtual/libx11 libxext libxrandr libxrender', '', d)} \ tslib"
> > > 
> > >  DEPENDS_class-nativesdk = "${@base_contains('DISTRO_FEATURES', 'x11',
> > > 
> > > 'virtual/nativesdk-libx11 nativesdk-libxrandr nativesdk-libxrender
> > > nativesdk-libxext', '', d)}" +DEPENDS_class-native =
> > > "${@base_contains('DISTRO_FEATURES', 'x11', 'libx11-native
> > 
> > libxrandr-native
> > 
> > > libxrender-native libxext-native', '', d)}"
> > > 
> > >  PR = "r2"
> > > 
> > > @@ -63,4 +64,4 @@ do_configure_prepend() {
> > > 
> > >          export SYSROOT=$PKG_CONFIG_SYSROOT_DIR
> > >  
> > >  }
> > > 
> > > -BBCLASSEXTEND = "nativesdk"
> > > +BBCLASSEXTEND = "nativesdk native"
> > 
> > Why do you want to do this? Usually, it's better to rely on the host
> > distro providing SDL for native because building libsdl-native can lead to
> > problems linking against a specific host OpenGL implementation.
> 
> xbmc needs sdl-native - this is why I made this change. 

Why does it need it though?

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre