On Tue, Nov 08, 2011 at 11:15:56AM -0800, Saul Wold wrote:
> On 11/07/2011 06:19 AM, Martin Jansa wrote:
> > * virtual/libsdl is used in sdl.bbclass but nothing was providing it
> > * rpath disabled because otherwise libtool and ld try to link against
> >    ${libdir}/libdl.so.2 on the host
> >    (see OE-classic commit af1a2e0e7626e372f22afbcabf08d9ae6b0d7b01)
> >
> > Signed-off-by: Martin Jansa<[email protected]>
> > ---
> >   meta/recipes-graphics/libsdl/libsdl_1.2.14.bb |   18 ++++++++++++------
> >   1 files changed, 12 insertions(+), 6 deletions(-)
> >
> > diff --git a/meta/recipes-graphics/libsdl/libsdl_1.2.14.bb 
> > b/meta/recipes-graphics/libsdl/libsdl_1.2.14.bb
> > index 6b23344..17a3103 100644
> > --- a/meta/recipes-graphics/libsdl/libsdl_1.2.14.bb
> > +++ b/meta/recipes-graphics/libsdl/libsdl_1.2.14.bb
> > @@ -10,10 +10,12 @@ SECTION = "libs"
> >   LICENSE = "LGPLv2.1"
> >   LIC_FILES_CHKSUM = "file://COPYING;md5=27818cd7fd83877a8e3ef82b82798ef4"
> >
> > -DEPENDS = "${@base_contains('DISTRO_FEATURES', 'opengl', 'virtual/libgl', 
> > '', d)} virtual/libx11 libxext libxrandr libxrender"
> > +PROVIDES = "virtual/libsdl"
> > +
> > +DEPENDS = "${@base_contains('DISTRO_FEATURES', 'opengl', 'virtual/libgl', 
> > '', d)} virtual/libx11 libxext libxrandr libxrender alsa-lib tslib"
> >   DEPENDS_virtclass-nativesdk = "libx11-nativesdk libxrandr-nativesdk 
> > libxrender-nativesdk libxext-nativesdk"
> >
> Shouldn't the enabling of alsa be based on the DISTRO_FEATURE of alsa?

OK

I'll send patch changing this and opengl to PACKAGECONFIG options.

> 
> Sau!
> 
> > -PR = "r0"
> > +PR = "r1"
> >
> >   SRC_URI = "http://www.libsdl.org/release/SDL-${PV}.tar.gz \
> >        file://configure_tweak.patch \
> > @@ -26,25 +28,29 @@ SRC_URI[sha256sum] = 
> > "5d927e287034cb6bb0ebccfa382cb1d185cb113c8ab5115a0759798642
> >
> >   inherit autotools binconfig pkgconfig
> >
> > -EXTRA_OECONF = "--disable-static --disable-debug --disable-cdrom 
> > --enable-threads --enable-timers --enable-endian \
> > -                --enable-file --disable-oss --disable-alsa --disable-esd 
> > --disable-arts \
> > +EXTRA_OECONF = "--disable-static --disable-debug --enable-cdrom 
> > --enable-threads --enable-timers --enable-endian \
> > +                --enable-file --disable-oss --enable-alsa --disable-esd 
> > --disable-arts \
> >                   --disable-diskaudio --disable-nas --disable-esd-shared 
> > --disable-esdtest \
> >                   --disable-mintaudio --disable-nasm --enable-video-x11 
> > --disable-video-dga \
> >                   --disable-video-fbcon --disable-video-directfb 
> > --disable-video-ps2gs --disable-video-ps3 \
> >                   --disable-video-xbios --disable-video-gem 
> > --disable-video-dummy \
> > -                --enable-input-events --enable-pthreads \
> > +                --enable-input-events --enable-input-tslib 
> > --enable-pthreads \
> >             ${@base_contains('DISTRO_FEATURES', 'opengl', 
> > '--enable-video-opengl', '--disable-video-opengl', d)} \
> >             --disable-video-svga \
> > -                --disable-video-picogui --disable-video-qtopia 
> > --enable-dlopen"
> > +                --disable-video-picogui --disable-video-qtopia 
> > --enable-dlopen \
> > +                --disable-rpath"
> >
> >   PARALLEL_MAKE = ""
> >
> > +EXTRA_AUTORECONF += "--include=acinclude --exclude=autoheader"
> > +
> >   do_configure_prepend() {
> >           # Remove old libtool macros.
> >           MACROS="libtool.m4 lt~obsolete.m4 ltoptions.m4 ltsugar.m4 
> > ltversion.m4"
> >           for i in ${MACROS}; do
> >                  rm -f acinclude/$i
> >           done
> > +        export SYSROOT=$PKG_CONFIG_SYSROOT_DIR
> >   }
> >
> >   BBCLASSEXTEND = "nativesdk"

-- 
Martin 'JaMa' Jansa     jabber: [email protected]

Attachment: signature.asc
Description: Digital signature

_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core

Reply via email to