[PATCH] Fix static linking with tremor & libogg

Bernd Kuhls <[email protected]>
Newsgroups gmane.comp.video.mplayer.devel
Message-ID <[email protected]>
_______________________________________________
MPlayer-dev-eng mailing list
[email protected]
https://lists.mplayerhq.hu/mailman/listinfo/mplayer-dev-eng
0005-tremor-ogg.patch (text/x-diff, 1.1 KB)
Fix static linking with tremor & libogg

The order of the libraries needs to be changed to fix a bug during
static linking caught by buildroot autobuilders:

http://autobuild.buildroot.net/results/bc9/bc98fa585399b53ea181dbaf392b93424145911d/

/home/test/autobuild/instance-3/output/host/usr/powerpc-buildroot-linux-uclibc/sysroot/usr/lib/libvorbisidec.a(synthesis.o): In function `_vorbis_synthesis1':
synthesis.c:(.text+0x90): undefined reference to `oggpack_readinit'
synthesis.c:(.text+0x9c): undefined reference to `oggpack_read'
[...]

Signed-off-by: Bernd Kuhls <[email protected]>

Index: configure
===================================================================
--- a/configure	(revision 37873)
+++ b/configure	(working copy)
@@ -6629,7 +6629,7 @@
   def_tremor='#define CONFIG_TREMOR 1'
   codecmodules="tremor $codecmodules"
   res_comment="integer libvorbis"
-  extra_ldflags="$extra_ldflags -logg -lvorbisidec"
+  extra_ldflags="$extra_ldflags -lvorbisidec -logg"
 elif test "$_libvorbis" = yes ; then
   _vorbis=yes
   def_vorbis='#define CONFIG_OGGVORBIS 1'
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.