[MPlayer-dev-eng] [patch] check pkg-config first for libdvd{nav, read}

[Mike Frysinger]

On Saturday 07 April 2012 05:01:38 Diego Biurrun wrote:
> On Fri, Mar 30, 2012 at 02:18:04PM -0400, Mike Frysinger wrote:
> > managing pkg-config files is a lot easier when cross-compiling then a
> > variety of config scripts.  the libdvd{nav,read} packages install .pc
> > files now, so check those first before falling back to the config
> > scripts.
> > 
> > (please cc me as i'm not on the list)
> > 
> > --- configure	(revision 34835)
> > +++ configure	(working copy)
> > @@ -4042,7 +4042,7 @@ echores "$_apple_ir"
> > 
> >  echocheck "pkg-config"
> > 
> > -_pkg_config=pkg-config
> > +_pkg_config=${PKG_CONFIG:-pkg-config}
> 
> That doesn't quite fit the patch description.

the build should respect the env $PKG_CONFIG just like the upstream pkg-config 
intended (see pkg.m4) which makes it work like every autotools based pkg out 
there.

at least the mplayer2 project has adopted a --pkg-config flag it seems.

> > @@ -7721,13 +7726,20 @@ if test "$_dvdnav" = auto ; then
> >      dvdnav_internal=yes
> >      res_comment="internal"
> >    else
> > -    $_dvdnavconfig --version --minilibs >> $TMPLOG 2>&1 || _dvdnav=no
> > +    if ! $_pkg_config --exists dvdnavmini ; then
> > +      $_dvdnavconfig --version --minilibs >> $TMPLOG 2>&1 || _dvdnav=no
> > +    fi
> >    fi
> >  fi
> 
> I wonder why you do not test pkg-config, but only dvdnav-config here.

not sure what you mean.  the logic is testing for the dvdnav .pc before 
falling back to the dvdnavconfig
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20120407/efb6100e/attachment.asc>