Bug 387

Summary:	Tremor not installed to toolchain
Product:	buildroot	Reporter:	Evan Zelkowitz <evan.zelkowitz>
Component:	Other	Assignee:	unassigned
Status:	RESOLVED FIXED
Severity:	normal	CC:	buildroot
Priority:	P5
Version:	unspecified
Target Milestone:	2010.05
Hardware:	All
OS:	Linux
Host:	linux	Target:	mipsel-linux
Build:

Description Evan Zelkowitz 2009-06-09 15:22:03 UTC

When selecting tremor usage for vorbis it installs into the root fs properly but it is not being installed into the toolchain, so everything still gets built against libvorbis instead of tremor.

Comment 1 Peter Korsgaard 2009-07-16 19:49:15 UTC

Thanks, fixed in git (bd26b581)

Comment 2 Sven Neumann 2009-07-21 12:00:25 UTC

This breaks the build of gst-plugins-base. Looks like there's a fix needed there as well:

make: *** No rule to make target `libvorbis', needed by `gst-plugins-base-depends'.  Stop.

Comment 3 Peter Korsgaard 2009-07-22 06:02:30 UTC

Thanks for the heads up. Should be fixed by 11eb084fc79a75395.

Comment 4 Sven Neumann 2009-07-22 10:22:41 UTC

I am sorry, but it is not fixed:

Making all in vorbis
make[4]: Entering directory `/home/sven/git/buildroot/build_arm/gst-plugins-base-0.10.23/ext/vorbis'
  CC    vorbis.o
  CC    vorbisparse.o
  CC    vorbisenc.o
  CC    vorbisdec.o
  CC    vorbistag.o
  LINK  libgstvorbis.la
/usr/lib/libvorbisenc.so: could not read symbols: File in wrong format
collect2: ld returned 1 exit status

gst-plugins-base tries to link against libvorbisenc.so and as far as I can see this library is not provided by tremor.

Comment 5 Peter Korsgaard 2009-07-22 15:02:53 UTC

(In reply to comment #4)
> I am sorry, but it is not fixed:

Ahh, sorry - I didn't explicitly enable libvorbis support when I did a test compile of gst-plugin-base, so I missed that.

> gst-plugins-base tries to link against libvorbisenc.so and as far as I can see
> this library is not provided by tremor.

No, tremor is only an implementation of the decoding API. We should either make the stuff needing the encoding API depend on !BR2_PACKAGE_LIBVORBIS_TREMOR or split up the libvorbis handling in encoding and decoding support, so you can use the normal (FPU using) libvorbis stuff for encoding and tremor for decoding (which I suspect is what you were doing before by luck as the tremor stuff was listed below libvorbis so it was overwriting the FPU version of the lib).

Comment 6 Peter Korsgaard 2009-07-22 15:04:05 UTC

(In reply to comment #4)

> /usr/lib/libvorbisenc.so: could not read symbols: File in wrong format
> collect2: ld returned 1 exit status

FYI, the reference to /usr/lib is because of a bug in pkg-config's sysroot support which I'm fixing now.

Comment 7 Peter Korsgaard 2010-07-18 20:35:36 UTC

I believe this is resolved by Svens addition of the BR2_PACKAGE_GST_PLUGINS_BAD_PLUGIN_TREMOR option (63dd44dbcdc6eb8) - If not, please reopen the bug.