[gstreamer-bugs] [Bug 539601] [realvideodec] not-negotiated error if codec lib isn't found

[GStreamer (bugzilla.gnome.org)]

If you have any questions why you received this email, please see the text at
the end of this email. Replies to this email are NOT read, please see the text
at the end of this email. You can add comments to this bug at:
  http://bugzilla.gnome.org/show_bug.cgi?id=539601

  GStreamer | gst-plugins-bad | Ver: HEAD CVS

Tim-Philipp Müller changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEEDINFO                    |RESOLVED
          Component|gst-plugins-ugly            |gst-plugins-bad
         Resolution|                            |FIXED
            Summary|[rmdemux] Fails negotiation |[realvideodec] not-
                   |                            |negotiated error if codec
                   |                            |lib isn't found
   Target Milestone|HEAD                        |0.10.8




------- Comment #6 from Tim-Philipp Müller  2008-06-24 15:17 UTC -------
I think what happens is: realvideodec is plugged since it claims to support all
real video versions, and then when the caps are set once it gets some data it
finds that the required codec lib is missing and errors out. Which is a bit
broken really, since it should only advertise the formats it can support from
the start.

I think this is fixed in -bad CVS though:

2008-06-13  Wim Taymans  <wim.taymans at collabora.co.uk>

        * gst/real/gstrealaudiodec.c: (gst_real_audio_dec_chain),
        (close_library), (open_library),
        (gst_real_audio_dec_probe_modules), (gst_real_audio_dec_getcaps),
        (gst_real_audio_dec_setcaps), (gst_real_audio_dec_init),
        (gst_real_audio_dec_change_state), (gst_real_audio_dec_finalize):
        Add raversions we can support on the caps.
        Refactor the loading of the real codecs like realvideo so that we can
        implement probing.
        Probe all supported formats by trying to load the .so files, only
report
        the versions on the caps that we can actually load.

        * gst/real/gstrealvideodec.c: (gst_real_video_dec_chain),
        (gst_real_video_dec_getcaps), (gst_real_video_dec_setcaps),
        (open_library), (close_library),
        (gst_real_video_dec_probe_modules),
        (gst_real_video_dec_change_state), (gst_real_video_dec_init),
        (gst_real_video_dec_finalize), (gst_real_video_dec_class_init):
        * gst/real/gstrealvideodec.h:
        Change the loading of the library like the audio decoder.
        Probe the supported formats by trying to load the .so files and only
        report the versions on the caps that we can actually load.


-- 
See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received
this email, why you can't respond via email, how to stop receiving
emails (or reduce the number you receive), and how to contact someone
if you are having problems with the system.

You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=539601.