[MPlayer-dev-eng] [PATCH] vdpau

[Gwenole Beauchesne]

On Mon, 16 Feb 2009, Reimar Doeffinger wrote:

>> Ah, didn't notice it was the new way to operate. Why is it so?
>
> Because vdpau is a proprietary library, thus most distributions will have to
> (and should!) compile MPlayer without vdpau-support unless it is dlopened, or
> in short: see the libdvdcss case (though we do not have the dlopen in MPlayer
> itself, it is in the external libdvdread).

Yes, that makes sense but I think I read somewhere nvidia would try to 
change at least the license for the wrapper library for this very reason. 
Though I don't remember where, was it a dream?

>> Besides, dlopen() an *.so is generally wrong as it implies *-dev{,evel}
>> packages at a distribution level...
>
> I will have to check, but again, VDPAU is proprietary and so far I am not
> aware of the nvidia drivers ever been split into a -dev package (I admit I
> mainly use gentoo though).

Debian and/or Ubuntu and probably even other RPM-based distributions 
(Mandriva?).

>> Beyond the fact that if their is an
>> API change (and thus, a SONAME change), you'd pick up the wrong library.
>
> The API used via dlopen/dlsym consists of _exactly_ one single function,
> vdp_device_create_x11.

which could have more or less args in the future, so don't assume anything 
to be on the safe side. At the very least, add the version number.