[vlc-devel] New LIVE555 version - should eliminate most "live*patch"s

[Jean-Paul Saman]

Jean-Baptiste Kempf wrote:
> On Fri, Jan 04, 2008, Ross Finlayson wrote :
>> (As I've noted in the past, I'd really like to see us mandate a 
>> minimum required LIVE555 version for "live555.cpp", so we can remove 
>> many of those ugly
>> 	#if (LIVEMEDIA_LIBRARY_VERSION_INT
>> lines.)
> 
> ok. jpsaman? advice ?
> 
> What about considering this version of live as NECESSARY to build vlc
> trunk ?

It will force everybody to update it's (contrib) live555 build for 
trunk. I would expect questions and bugreports on the mailinglist if we 
do this (especially when people forget to remove the live-latest tarball 
from contrib before rebuilding). In summary it breaks backwards 
compatibility.

I understand Ross his request, it will make the code cleaner. VLC policy 
has always been to keep backwards compatibility with external libraries 
(as long as it makes sense to do so). This is especially important for 
dynamic loaded libs. However since live is statically linked we could 
consider making latest live555 version mandatory and check for the 
version number in configure-step.

Is anybody in the team against this approach?

Gtz,
Jean-Paul Saman.