Opened 13 years ago

Closed 13 years ago

#1878 closed defect (fixed)

mplayer fails to build with libvorbis support

Reported by: DummyUnit@… Owned by: diego@…
Priority: normal Component: build system
Version: HEAD Severity: normal
Keywords: Cc: cehoyos
Blocked By: Blocking:
Reproduced by developer: no Analyzed by developer: no

Description

Trying to build mplayer with Vorbis support via external libvorbis instead of internal tremor fails on linking stage with following error:

ffmpeg/libavcodec/libavcodec.a(libvorbis.o): In function `oggvorbis_encode_init':
libvorbis.c:(.text.unlikely+0xb5): undefined reference to `vorbis_encode_setup_vbr'
libvorbis.c:(.text.unlikely+0xef): undefined reference to `vorbis_encode_setup_managed'
libvorbis.c:(.text.unlikely+0x112): undefined reference to `vorbis_encode_ctl'
libvorbis.c:(.text.unlikely+0x14a): undefined reference to `vorbis_encode_ctl'
libvorbis.c:(.text.unlikely+0x170): undefined reference to `vorbis_encode_ctl'
libvorbis.c:(.text.unlikely+0x178): undefined reference to `vorbis_encode_setup_init'

Steps to reproduce:

  1. ./configure --disable-tremor-internal
  2. make

Tested on latest git (at moment of reporting): b1c5fec98d73710dc10b15a9c342f12589ce5d2f

Adding -lvorbisenc to linking command fixes this error. There is some patch to configure that fixes this problem: http://bugs.gentoo.org/show_bug.cgi?id=344593#c29

Change History (4)

comment:1 by cehoyos, 13 years ago

Cc: cehoyos@… added

Work-around is to use --disable-mencoder.

comment:2 by DummyUnit@…, 13 years ago

(In reply to comment #1)

Work-around is to use --disable-mencoder.

Unfortunately this bug affects mplayer itself.
BTW For me, this bug is already solved with this dirty hack:

diff --git a/configure b/configure
index 1f59faf..912a350 100755
--- a/configure
+++ b/configure
@@ -6319,7 +6319,7 @@ elif test "$_libvorbis" = yes ; then

def_vorbis='#define CONFIG_OGGVORBIS 1'
codecmodules="libvorbis $codecmodules"
res_comment="libvorbis"

  • extra_ldflags="$extra_ldflags -lvorbis -logg"

+ extra_ldflags="$extra_ldflags -lvorbis -logg -lvorbisenc"

libavencoders="$libavencoders LIBVORBIS_ENCODER"

else

_vorbis=no

It works, but probably it's not a correct fix.

comment:3 by DummyUnit@…, 13 years ago

(In reply to comment #2)

(In reply to comment #1)

Work-around is to use --disable-mencoder.

Unfortunately this bug affects mplayer itself.

But --disable-mencoder also fixes this problem, strange...

comment:4 by DummyUnit@…, 13 years ago

Resolution: fixed
Status: newclosed

Fixed in git:91857841e2c8502c23e0416a50a3af2d07c7e391

Note: See TracTickets for help on using tickets.