After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 710255 - lamemp3enc ! flvmux - is not linking
lamemp3enc ! flvmux - is not linking
Status: RESOLVED NOTABUG
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
1.x
Other Linux
: Normal normal
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2013-10-16 09:35 UTC by Harish Jenny K N
Modified: 2013-10-17 20:01 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
patch to fix the bug (1.34 KB, patch)
2013-10-16 09:35 UTC, Harish Jenny K N 		rejected Details | Review

Description Harish Jenny K N 2013-10-16 09:35:00 UTC 
gst-launch-1.0 audiotestsrc ! lamemp3enc ! flvmux ! fakesink not working


Setting pipeline to PAUSED ...
Pipeline is PREROLLING ...
Redistribute latency...
ERROR: from element /GstPipeline:pipeline0/GstAudioTestSrc:audiotestsrc0: Internal data flow error.
Additional debug info:
gstbasesrc.c(2865): gst_base_src_loop (): /GstPipeline:pipeline0/GstAudioTestSrc:audiotestsrc0:
streaming task paused, reason not-negotiated (-4)
ERROR: pipeline doesn't want to preroll.
Setting pipeline to NULL ...
Freeing pipeline ...
Comment 1 Harish Jenny K N 2013-10-16 09:35:30 UTC 
Created attachment 257408 [details] [review]
patch to fix the bug

patch for fix the bug
Comment 2 Sebastian Dröge (slomo) 2013-10-16 10:33:45 UTC 
You have to add mpegaudioparse between encoder and muxer. FLV requires to have one MP3 frame per FLV packet, thus requires framed/parsed input.
Comment 3 Harish Jenny K N 2013-10-16 11:06:26 UTC 
framed: true was removed from sink caps of FLVmux inorder to connect faac ! flvmux from this commit 
http://cgit.freedesktop.org/gstreamer/gst-plugins-good/commit/?id=29effb522a91986ebf96c60e0c6ed0de6ffc385d

Does flv also not require one AAC frame per FLV packet ? Why is framed=true was removed from sink caps ?
Comment 4 Sebastian Dröge (slomo) 2013-10-16 11:11:21 UTC 
Because stream-format=raw already implies framed input. For MP3 there's nothing that implies it.
Comment 5 Harish Jenny K N 2013-10-16 11:39:03 UTC 
Thanks for the update !
Does not QTmux also expect parsed=true in case of mp3 data for various container formats?

Because I see 
gst-launch-1.0 audiotestsrc ! lamemp3enc ! qtmux ! filesink location=~/sample.mp4

working.
Comment 6 Sebastian Dröge (slomo) 2013-10-17 20:01:55 UTC 
I think so, but not 100% sure right now. Would make sense to file another bug about that.

I'm sure it's also wrong in some other muxers.