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 613215 - delayed set in gst-launch are not recursive
delayed set in gst-launch are not recursive
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gstreamer (core)
unspecified
Other Linux
: Normal normal
: 0.10.29
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2010-03-18 08:34 UTC by Stefan Sauer (gstreamer, gtkdoc dev)
Modified: 2010-03-22 09:56 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
make delayed set recursive (1.97 KB, patch)
2010-03-18 08:35 UTC, Stefan Sauer (gstreamer, gtkdoc dev) 		needs-work Details | Review
make delayed set recursive (6.81 KB, patch)
2010-03-18 12:29 UTC, Stefan Sauer (gstreamer, gtkdoc dev) 		none Details | Review

Description Stefan Sauer (gstreamer, gtkdoc dev) 2010-03-18 08:34:32 UTC 
playbin2 uri=rtsp://www.server.com/movie.mpg
uridecodebin0::source::debug=true

does not enable debug=true
Comment 1 Stefan Sauer (gstreamer, gtkdoc dev) 2010-03-18 08:35:13 UTC 
Created attachment 156435 [details] [review]
make delayed set recursive
Comment 2 Stefan Sauer (gstreamer, gtkdoc dev) 2010-03-18 09:22:51 UTC 
Comment on attachment 156435 [details] [review]
make delayed set recursive

the patch is not enough. Another usecase:

gst-launch-0.10 playbin2 uri=file://movie.avi playsink0::vbin::videosink::videosink-actual-sink-xvimage::force-aspect-ratio=true
Comment 3 Stefan Sauer (gstreamer, gtkdoc dev) 2010-03-18 12:29:10 UTC 
Created attachment 156445 [details] [review]
make delayed set recursive

This works for me.
Comment 4 Sebastian Dröge (slomo) 2010-03-18 19:30:45 UTC 
Looks good I guess but for this to be really useful we should start to use hardcoded element names inside playbin2, etc.
Comment 5 Stefan Sauer (gstreamer, gtkdoc dev) 2010-03-22 09:56:35 UTC 
playbin2 is not the only application. Anyway better fix it than leave it broken.

commit da287f556db94f0c1a1ad16315a0a02c3f0e2b8f
Author: Stefan Kost <ensonic@users.sf.net>
Date:   Thu Mar 18 10:22:09 2010 +0200

    parse-launch: make delayed set recursive
    
    Right now deleyed set would only try for first set of children. We need to keep
    trying to support arbitrary deep hierarchies (like in playbin2 with auto*sinks).
    Also GstBin would need to actualy emit the child-added/removed signal as it
    implements the iface. Fixes #613215.