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 120065 - Support playing and stopping in gst-launch via signals.
Support playing and stopping in gst-launch via signals.
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gstreamer (core)
git master
Other Linux
: Normal enhancement
: 0.7.x
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2003-08-17 11:59 UTC by ramon
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Proposed gst-launch. (9.89 KB, text/plain)
2003-08-17 12:01 UTC, ramon
  Details
Diff against CVS. (2.12 KB, text/plain)
2003-08-18 21:44 UTC, Ramon Garcia Fernandez
  Details
Updated diff (2.71 KB, patch)
2003-08-19 05:34 UTC, Ronald Bultje
none Details | Review

Description ramon 2003-08-17 11:59:44 UTC
gst-launch is useful for debugging elements. Thus it does make sense to have
functionality that allows one to play and stop the pipeline. In this way
the developer of a plugin can test the behaviour of an element under state
change.
Comment 1 ramon 2003-08-17 12:01:42 UTC
Created attachment 19285 [details]
Proposed gst-launch.
Comment 2 ramon 2003-08-17 12:03:27 UTC
The attached source implements the following behaviour:

- SIGUSR1 changes state to PLAYING.
- SIGUSR2 changes state to NULL.

Others could be added, but these are IHMO the most important for testing.
Comment 3 Ronald Bultje 2003-08-17 19:00:58 UTC
Isn't gst-launch-gui (from the gst-editor package) much more useful
for this? It provides the same functionality as gst-launch, but in a
UI (Gnome-2 based), including stop/pause/play buttons...

Or gst-editor itself... It provides support for quite some features.

Isn't it a bit overkill to add this to gst-launch?
Comment 4 ramon 2003-08-18 09:57:41 UTC
The most important advantadge for me of gst-launch is that, since the
code is simpler, it is better for debugging with gdb. With this patch,
I can put a breakpoing with gdb in play_handler, follow it and learn
why the request is not working.
Comment 5 Ronald Bultje 2003-08-18 10:15:26 UTC
Hm, ok, I see... Sounds interesting. :). Could you add a diff -u patch
against CVS?
Comment 6 Ramon Garcia Fernandez 2003-08-18 21:44:34 UTC
Created attachment 19320 [details]
Diff against CVS.
Comment 7 Ronald Bultje 2003-08-19 05:34:10 UTC
Created attachment 19337 [details] [review]
Updated diff
Comment 8 Ronald Bultje 2003-08-19 05:35:29 UTC
The above patch:
1) Fixes compilation with -Werror turned on for your patch
2) Moves over if/else if/else loops to switch/case
3) Adapts coding style to our default

This looks fine to apply to me. :).
Comment 9 Ronald Bultje 2003-08-19 05:40:52 UTC
Applied. Thanks for the idea. :).
Comment 10 David I. Lehn 2003-08-19 05:50:48 UTC
This patch is incomplete.  The man page needs to be updated to reflect
these new capabilities.  Please reopen until this is done.
Comment 11 David I. Lehn 2003-08-19 05:53:20 UTC
Doh!  I should have waited for the CVS log to come through.  I see
docs were updated.  Nevermind last comment.