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

[Nico Sabbi]

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.
> 
> Diego

because of 2+1 reasons it's not yet always feasible:
- this particular configuration can be enabled only with the
   new build system, that doesn't work on some platforms (e.g. osx);
   the build system desperately needs an expert maintainer, that I'm 
  not.
   Any volunteer ? :-))))
- by development policy libdvdnav must be backwards compatible,
   so removing dvdread tout-court is out of question
- it's still possible to have 2 different versions of dvdnav
   (one backwards compatible for vlc/xine/whatever and another one
   without dvdread for us), but I doubt that users want to have
   2 different versions of the library