improving vlc-devel

[Michael Johnson]

On 2/26/07, Mikhail Teterin <mi+kde at aldan.algebra.com> wrote:
>
> Hello!
>
> Attached is a patch, that fixes two minor annoyances with the port:
>
>         . removes "BROKEN" on amd64 -- just built it fine on my system;
>         . removes explicit requirements for shared libraries' major
> numbers.
>
> There are several commits in the Makefile's cvs log, which do nothing
> but "chase" these numbers -- that's a foolish excercise. Vlc (and most
> other
> software) almost never care for a particular version of a dependency and
> the
> shlib major number should only be specified explicitly, when needed --
> ommited _by default_ (vlc's own configure-script is better at detecting
> such
> incompatibilities anyway).


That's not the reason we use LibName.LibVersion in VLC. VLC and many other
programs
hard link to LibName.LibVersion, we keep the LibVersion in there so it
reduces the
amount of bug reports we get.

Here is an example if libx264.so.50 changes to libx264.so.99 (I chose x264
since we patch
it to control the LibVersion number to reduce the number of rebuilds on VLC
and mplayer)

root at blueherron /usr/ports/multimedia/x264 > mplayer -1
MPlayer 1.0rc1-3.4.6 (C) 2000-2006 MPlayer Team
CPU: Unknown CPU Type (Family: 6, Model: 10, Stepping: 0)
CPUflags:  MMX: 1 MMX2: 1 3DNow: 1 3DNow2: 1 SSE: 1 SSE2: 0
Compiled with runtime CPU detection.
Unknown option on the command line: -1
root at blueherron /usr/ports/multimedia/x264 > ldconfig -r|grep x264.so
        426:-lx264.50 => /usr/local/lib/libx264.so.50

<<<<snipped out the reinstall of x264>>>>

root at blueherron /usr/ports/multimedia/x264 > ldconfig -r|grep x264.so
        426:-lx264.99 => /usr/local/lib/libx264.so.99
root at blueherron /usr/ports/multimedia/x264 > mplayer
/libexec/ld-elf.so.1: Shared object "libx264.so.50" not found, required by
"mplayer"

even though this is mplayer, vlc does the same thing.

I'm open to a patch that makes VLC link to LibName.so instead of
LibName.so.LibVersion.

Michael

Doing otherwise requires too many rebuilds to make sense -- in order to
> build
> a fresh version of vlc, for example, one has to rebuild gaim too, because
> vlc
> explicitly requires gnutls.15, whereas one's existing gaim install may
> refer
> to gnutls.12...
>
> Please, consider the attached patch. Thank you very much for your time,
>
>         -mi
>
>