Re: New vlc 1.1.6

On Sun, 2011-01-30 at 13:27 -0200, Paulo Cavalcanti wrote:
> On Sun, Jan 30, 2011 at 12:10 PM, Axel Thimm <[email protected]>
> wrote:
> On Sun, 2011-01-30 at 10:54 -0200, Paulo Cavalcanti wrote:
> > Finally, you disabled musepack-tools-devel because mplayer does not
> > compile with it.
>         I didn't disable it, it is still there. What I did was to
>         defuse the drop-in replacement of libmpcdec-devel since
>         musepack is neither claiming to be API, nor ABI backward
>         compatible to libmpcdec. Looking at the sources it looks like
>         the developers even intend for users to be able to coinstall
>         both.

> You commented the include bits:
> #%{_includedir}/mpcdec

Yes, that was just a symlink from mpc/*.h (sv8) to mpcdec/*.h (1.2.6).
sv8's include files are not backward compatible and therefore upstream
changed the include path deliberately for the application to catch up.
Symlinking the old location defeats that purpose.

> Vlc compiles fine with sv8. 

Yes, but it explicitly checks both positions and prefers sv8's mpc/*.h,
if found. I don't think there is any not ported application that just
builds by symlinking the include trees, at least not vlc and mplayer.

-- 
http://thimm.gr/ - http://ATrpms.net/

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)

iEYEABECAAYFAk1FqzwACgkQQBVS1GOamfFMJQCff+LVZFISI4UycS5bl3sHDpbH
F9sAni6KhRBgffqqKQ5adZaD8hiRDY5L
=M+rO
-----END PGP SIGNATURE-----