[MPlayer-dev-eng] DirectFB configure fix

Ville Syrjälä syrjala at sci.fi
Fri Dec 22 07:07:44 CET 2006


On Thu, Dec 21, 2006 at 08:21:45PM +0100, Benjamin Zores wrote:
> On Thu, 21 Dec 2006 20:36:20 +0200
> Ville Syrjälä <syrjala at sci.fi> wrote:
> 
> > On Thu, Dec 21, 2006 at 07:09:11PM +0100, Benjamin Zores wrote:
> > > Hi,
> > > 
> > > Attached patch adds new option to configure to specify directfb-config path
> > > (same way it's done for other *-config binaries).
> > > It allows using it to retrieve DirectFB version, but also cflags and libs which
> > > allows cross-compilation by removing hardcoded include paths.
> > > 
> > > Ok to apply ?
> > 
> > No.
> > 
> > <snip>
> > > +  if "$_directfbconfig" --version >>"$TMPLOG" 2>&1 ; then
> > > +    if cc_check `$_directfbconfig --cflags` `$_directfbconfig --libs` >>"$TMPLOG" 2>&1 ; then
> > > +      _directfbversion=`$_directfbconfig --version | sed 's/[^0-9]//g'`
> > > +       if test "$_directfbversion" -gt 0913 ; then
> > 
> > This will break the version check again. Use major/minor/micro and 
> > dfb_version() like the current code.
> 
> It it used later on (see the patch) to determine the #define value, used in vo sources.

You need to use it here too.

-- 
Ville Syrjälä
syrjala at sci.fi
http://www.sci.fi/~syrjala/



More information about the MPlayer-dev-eng mailing list