Summary: | media-plugins/gst-plugins-flac -- flac 1.1.3 compatibility fix | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Alexis Ballier <aballier> |
Component: | Current packages | Assignee: | GStreamer package maintainers <gstreamer> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | gnome, sound, tom |
Priority: | High | ||
Version: | 2006.1 | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | |||
Bug Blocks: | 152007 | ||
Attachments: |
Patch from Josh Coalson
Configure patch |
Description
Alexis Ballier
2006-12-10 02:53:22 UTC
Created attachment 103734 [details, diff]
Patch from Josh Coalson
I tried your patch, but somehow the plugin is not liked against libFLAC.so and thus gstreamer fails to load it: 0:00:00.054136000 5149 0x50b830 WARN GST_PLUGIN_LOADING gstplugin.c:414:gst_plugin_load_file: module_open failed: /usr/lib64/gstreamer-0.10/libgstflac.so: undefined symbol: FLAC__StreamDecoderStateString If I add LD_PRELOAD=/usr/lib/libFLAC.so to the commandline then everything works fine. Created attachment 105683 [details, diff]
Configure patch
You're right, configure checks had not been updated for flac 1.1.3 and then it thought that flac will not be built. With the ebuild it builds it anyway but lacks the linking to libflac that configure has just not set because it couldn't detect flac.
Here is a patch to configure.ac that will make it check for the new and old flac api.
even with your second patch it doesn't work: configure: *** checking feature: FLAC lossless audio *** configure: *** for plug-ins: flac *** checking for FLAC__seekable_stream_encoder_new in -lFLAC... no configure: *** These plugins will not be built: flac it compiles, but I still have to use LD_PRELOAD. And I couldn't get configure to recognize the new flac (/usr/include/FLAC/export.h:#define FLAC_API_VERSION_CURRENT 8)
> it compiles, but I still have to use LD_PRELOAD. And I couldn't get configure
> to recognize the new flac (/usr/include/FLAC/export.h:#define
> FLAC_API_VERSION_CURRENT 8)
>
You probably forgot to call eautoconf / eautoreconf
(In reply to comment #5) > You probably forgot to call eautoconf / eautoreconf > Thanks, inherit autotools and calling eautoconf in src_compile() did the trick. Any news when this will this be merged into the official portage tree? Or do yo just wait for upstream to fix it in the next release? Adding this in gentoo's portage tree doesn't depend on me, that's up to gstreamer herd. According to http://bugzilla.gnome.org/show_bug.cgi?id=385887, this will be included in future releases, but I don't know which one nor when. I'm not pushing this migration because there are still some apps I'm supposed to fix. I opened those bugs to help maintainers / users to migrate, adding this to portage would help making the transition smooth but it's not a high priority problem. @gstreamer/gnome : ping, you're one of the last ones remaining for this migration, mind if I port & apply the patch to the latest release in the tree ? /me dons gnome hat Both patches look fine to me. I see your last message on gnome's bugzy dates back from January, don't let them forget this patch if they haven't committed it yet @gstreamer, package is still yours though, your call. :) Hmmm, ping? patch ported, merged in a single one, tested against flac 1.1.2 & 1.1.4 and commited. |