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

[SciFi]

On Sat, 28 Apr 2007 14:26:44 +0200, 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

Hi,

On a related note, I asked on the dvdnav list why did they not
choose to fork the libdvdnav2 (TWO) version.

ATPT the xine-lib project requires libdvdnav2, and we cannot have
your forked version installed here because of missing symbols when
linking xine-lib (1.1.6) against it due to your having chosen to
fork the non-2 version.

Hope that makes sense.  ;)

So the way I see it you *are* using the old libdvdnav version for
the fork.  And I say yes let's get rid of it to prevent these
conflicts, stop wasting time fixing this one, and get to work on
the libdvdnav2 tree which is already mirrored in your repo. 
Please.  ;)