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

[Andrei Gherzan <andrei@gherzan.ro>]

On Mon, Jan 13, 2014 at 02:09:36PM +0000, Paul Eggleton wrote:
> On Friday 03 January 2014 16:58:10 Andrei Gherzan wrote:
> > On Tue, Dec 17, 2013 at 7:01 PM, Andrei Gherzan <andrei@gherzan.ro> wrote:
> > > On Tue, Dec 17, 2013 at 6:53 PM, Paul Eggleton <
> > >
> > > paul.eggleton@linux.intel.com> wrote:
> > >> 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?
> > >
> > > Actually xbmc has a tool called texturepacker which uses libsdl-image -
> > > which needs libsdl. That's the chain.
> >
> > Any advice on this? Paul, do you see another option on this?
>
> To be honest I'm just not sure what the correct solution is here. The problem
> is if we just enable libsdl-native and take no other mitigating steps, it will
> almost certainly lead to problems building qemu-native because we'll be
> linking against our version of SDL and not the host's one.
>
> What does texturepacker actually do, i.e. what are its inputs and outputs? Is
> there a way of avoiding the need to build/use it?
>

As Alex was actually working on this, will let him give you (us) more details.

Alex, could you please provide Paul more info about that tool?

--
Andrei Gherzan
m: +40.744.478.414 | f: +40.31.816.28.12