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 544003 - [parse] Links to wrong pad if no padtemplate name is given
[parse] Links to wrong pad if no padtemplate name is given
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gstreamer (core)
git master
Other All
: Normal enhancement
: 0.10.22
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2008-07-21 14:38 UTC by Julien Isorce
Modified: 2008-12-09 16:12 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description Julien Isorce 2008-07-21 14:38:47 UTC 
Steps To Reproduce:

gst-launch-0.10 videotestsrc ! video/x-raw-yuv, width=720, height=576,format=(fourcc)I420 ! videoscale ! video/x-raw-yuv, width=400, height=300 ! textoverlay text="hello" ! fakesink

Actual Results:

WARNING: erroneous pipeline: could not link videoscale0 to textoverlay0

Expected Results:

Link ok.

Additional Information:

<__tim>	cap: it tries to link to textoverlay's text sink pad and when that fails it doesn't try the video pad
__tim>	cap: sth like this should work though: gst-launch-0.10 videotestsrc ! video/x-raw-yuv, width=720, height=576,format=\(fourcc\)I420 ! videoscale ! video/x-raw-yuv, width=400, height=300 ! overlay.video_sink textoverlay name=overlay text="hello" ! fakesink
Comment 1 Sebastian Dröge (slomo) 2008-07-21 17:36:56 UTC 
Ok so the problem is more, that gst_parse_launch and friends take a pad template at random if no name is given, try to link and explode if it fails. Instead it should try all possible pad templates until one works.
Comment 2 Julien Isorce 2008-07-21 19:28:17 UTC 
Hi, 

Note that gst_element_link_filtered (videoscale, textoverlay, caps) failed. 
And gst_element_link_pads_filtered(videoscale, "src", textoverlay,
"video_sink", caps) works.

J.
Comment 3 Wim Taymans 2008-10-13 14:06:12 UTC 
gst-launch is only a debug tool, closing. Please reopen if a patch is supplied.
Comment 4 Julien Isorce 2008-10-13 14:29:06 UTC 
? It comes from code too (comment #2)
Comment 5 Wim Taymans 2008-10-13 14:31:08 UTC 
ah, infamous gstutils functions that try to be too clever...
Comment 6 Sebastian Dröge (slomo) 2008-12-08 18:08:20 UTC 
*** Bug 563728 has been marked as a duplicate of this bug. ***
Comment 7 Christian Fredrik Kalager Schaller 2008-12-09 14:27:20 UTC 
Switching this bug over to 'enhancement'. While gst-launch is only a debug tool I think that expecting consistent behaviour of our muxers and demuxers when used through it is not to much to ask and different from asking for further features from gst-launch.
Comment 8 Sebastian Dröge (slomo) 2008-12-09 15:47:27 UTC 
2008-12-09  Sebastian Dröge  <sebastian.droege@collabora.co.uk>

        * gst/gstutils.c: (gst_element_get_compatible_pad):
        Check if the caps of the pads are compatible before returning
        a pad and claiming it is compatible. This, among other things,
        fixes a bug with gst-launch where an incompatible pad is chosen
        and linking fails. Fixes bug #544003.
Comment 9 Sebastian Dröge (slomo) 2008-12-09 16:12:04 UTC 
FYI Bug 563728 is actually no duplicate of this bug