[MPlayer-dev-eng] [PATCH] ddvnav+internal dvdread
diego at biurrun.de
Mon Apr 30 11:15:43 CEST 2007
On Mon, Apr 30, 2007 at 11:06:47AM +0200, Nico Sabbi wrote:
> Nico Sabbi wrote:
> > Diego Biurrun wrote:
> >>>@@ -7542,6 +7532,8 @@
> >>> _libavmuxers=`echo $_libavmuxers | sed -e s/LIBNUT_MUXER// `
> >>>+test "$_dvdnav" = yes && _ld_extra="$_ld_extra `$_dvdnavconfig --minilibs`"
> >>This is unacceptable, it belongs together with the libdvdnav test, don't
> >>spread things all over configure.
> > it's there for a reason: libdvdnavmini depends on dvdread code, but
> > it's not linked to libdvdread to permit mplayer to link either the
> > internal dvdread or the global library ( -ldvdread ) .
> > If I moved the assignment in the dvdnav checking block all the
> > following tests would be performed with $_ld_extra containing
> > -ldvdnavmini and would fail because of the lack of the dvdread symbols.
> > BTW, why do we keep on appending flags to $_ld_extra along the way
> > rather than only at the end?
Because often we depend on the output of some test etc. Also, it's
better to keep the tests in one piece.
> or, if you prefer, I can check the whole dvdnav checking code at the
> bottom of configure (as last test)
Just move the libdvdnav test after the libdvdread test and disable
libdvdnav if no libdvdread is available.
More information about the MPlayer-dev-eng