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 614805 - gst-launch: add --loop argument
gst-launch: add --loop argument
Status: RESOLVED WONTFIX
Product: GStreamer
Classification: Platform
Component: gstreamer (core)
git master
Other Linux
: Normal enhancement
: NONE
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2010-04-04 13:25 UTC by Rob Clark
Modified: 2013-07-24 10:56 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
add --loop argument (1.98 KB, patch)
2010-04-04 13:26 UTC, Rob Clark
needs-work Details | Review

Description Rob Clark 2010-04-04 13:25:33 UTC
if --loop is specified, automatically seek to beginning of clip when EOS is received

This is useful at least for demo's, because the video clip replays faster than it would if the gst-launch process exited and restarted (and re-auto-constructed the playbin2 pipeline).  This patch also exposed some problem in our codec elements.  So perhaps it is useful to others too?
Comment 1 Rob Clark 2010-04-04 13:26:02 UTC
Created attachment 157894 [details] [review]
add --loop argument
Comment 2 Tim-Philipp Müller 2010-04-04 13:44:20 UTC
I have a similar patch, but it does a state change cycle instead of a seek to 0 for the looping (useful to detect memory/resource leaks and the like).
Comment 3 Rob Clark 2010-04-07 13:17:20 UTC
(In reply to comment #2)
> I have a similar patch, but it does a state change cycle instead of a seek to 0
> for the looping (useful to detect memory/resource leaks and the like).

six of one, half dozen of the other ;-)

I guess either approach would detect different issues.. but then again a simple python script could do either too..

it's your call whether to merge one or the other (or both or neither)
Comment 4 Stefan Sauer (gstreamer, gtkdoc dev) 2011-02-28 15:14:15 UTC
Tim, would you like to push your patch then?
Comment 5 Sebastian Dröge (slomo) 2011-05-24 08:35:55 UTC
Maybe we should add something more generic like events syntax of gst-auto-launch instead:
https://github.com/tigrux/gst-auto-launch
Comment 6 Tim-Philipp Müller 2011-05-24 08:49:17 UTC
> Maybe we should add something more generic like events syntax of
> gst-auto-launch instead:  https://github.com/tigrux/gst-auto-launch

Yes, I think that'd be preferable too. However, I think we should default to reading 'commands' from a file or stdin line-by-line (with a line-ending separator maybe) instead of passing them as command line arguments. And the syntax could probably use some more refining, to make it more generic and extensible.
Comment 7 Stefan Sauer (gstreamer, gtkdoc dev) 2011-05-25 15:18:34 UTC
I like the idea of having a --commands=<filename> ('-' for stdin) and one command per line (\n as a command terminator). I think having a normal gst-timestamp (like in the log) would be good:
0:00:01.000000000 command <args>
0:00:02.200000000 command <args>

if the timestamp is ommitted, we could assume the same timestamp like prev line.

command could be:
state {null,ready,paused,playing}
event {seek, eos, .... }
query {duration, position, ...}
property element property name value
Comment 8 Stefan Sauer (gstreamer, gtkdoc dev) 2011-08-09 09:42:08 UTC
Again something similar was requested in Bug #612376.
Comment 9 Sebastian Dröge (slomo) 2013-07-24 08:17:51 UTC
Someone interested in working on this? Alternatively ges-launch should be able to do looping
Comment 10 Tim-Philipp Müller 2013-07-24 10:56:28 UTC
I think we need something like gst-launch, but more powerful, written from scratch with that in mind, maybe with some simple scripting foo. I think we should WONTFIX this, esp. if ges-launch can do it.