[MPlayer-dev-eng] [PATCH] ddvnav+internal dvdread

[Zuxy Meng]

Hi,

2007/4/30, Nico Sabbi <nicola_sabbi at fastwebnet.it>:
> Diego Biurrun wrote:
> > On Sat, Apr 28, 2007 at 02:48:45PM +0200, Nico Sabbi wrote:
> >
> >>Diego Biurrun wrote:
> >>
> >>>On Sat, Apr 28, 2007 at 02:10:03PM +0200, Nico Sabbi wrote:
> >>>
> >>>
> >>>>If you install my fork of dvdnav configured to use an external dvdread,
> >>>>with this patch applied mplayer can use dvdnav and mplayer's dvdread
> >>>>(unless you pass --disable-dvdread-internal, in which case the system-wide
> >>>>dvdread will be used).
> >>>
> >>>Why don't we drop support for the old libdvdnav versions?  AFAICT it
> >>>does not work anyway, so we should just require our forked version and
> >>>get rid of all these hacks.
> >>
> >
> >>- by development policy libdvdnav must be backwards compatible,
> >>   so removing dvdread tout-court is out of question
> >
> >
> > Why?  You just took over, make those big changes!
> >
>
> patch updated: now configure works only when dvdnav-config --minilibs
> works correctly (linking against libdvdnavmini.so), hence the old
> libdvdnav.so with dvdread embedded won't be used anymore.

libdvdnavmini.dll can't possibly be built under Windows; libdvdnav
can. So a Windows user can either build a static libdvdmini.a, or
disable the internal dvdread. Can we backport changes in
mplayer/dvdread to libdvdnav/src/dvdread so that libdvdnav.dll isn't
inferior to libdvdmini+internal dvdread?
-- 
Zuxy
Beauty is truth,
While truth is beauty.
PGP KeyID: E8555ED6