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 751901 - gst-play: verbose & playbin flags options support
gst-play: verbose & playbin flags options support
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
git master
Other Linux
: Normal enhancement
: 1.7.2
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2015-07-03 13:43 UTC by Hugues Fruchet
Modified: 2016-01-21 10:48 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
tools: gst-play: verbose & playbin flags options support (3.57 KB, patch)
2015-07-03 13:43 UTC, Hugues Fruchet
committed Details | Review

Description Hugues Fruchet 2015-07-03 13:43:19 UTC
Created attachment 306709 [details] [review]
tools: gst-play: verbose & playbin flags options support

Enhancements in gst-play in order to enable verbose playbin option and pass playbin flags.

Flags could be usefull to disable subtitle track by default for example:
 gst-play-1.0 "dvb://France 2 HD" --flags=0x13
Comment 1 Nicolas Dufresne (ndufresne) 2015-07-03 13:50:50 UTC
(In reply to Hugues Fruchet from comment #0)
> Created attachment 306709 [details] [review] [review]
> tools: gst-play: verbose & playbin flags options support
> 
> Enhancements in gst-play in order to enable verbose playbin option and pass
> playbin flags.
> 
> Flags could be usefull to disable subtitle track by default for example:
>  gst-play-1.0 "dvb://France 2 HD" --flags=0x13

I think we should support string from for the flags too, like we do in parse_launch(). I believe there is helpers for that. So we could:

  gst-play-1.0 --flags=video+audio

I really like this idea.
Comment 2 Tim-Philipp Müller 2016-01-18 19:46:47 UTC
I have merged most of this and added support for specifying flags by name as well. I did not merge the verbose exception stuff, because I'm not sure how useful it really is in the context of playbin (even in gst-launch it was broken for about 10 years without anyone noticing, so clearly not so important). Can always be added later if needed.

commit 4f5a263381d03073e633faa61ad6c1cf9816b097
Author: Tim-Philipp Müller <tim@centricular.com>
Date:   Thu Jan 14 23:14:27 2016 +0000

    tools: gst-play: allow setting of flags in serialized foo+bar format
    
    https://bugzilla.gnome.org/show_bug.cgi?id=751901

commit 1c72c6ddce80370b207e223425faec4db998f807
Author: Hugues Fruchet <hugues.fruchet@st.com>
Date:   Thu Jul 2 17:58:00 2015 +0200

    tools: gst-play: add command line options for verbose output and playbin flags
    
    https://bugzilla.gnome.org/show_bug.cgi?id=751901
Comment 3 Tim-Philipp Müller 2016-01-18 19:47:29 UTC
Comment on attachment 306709 [details] [review]
tools: gst-play: verbose & playbin flags options support

Pushed without the exclude list, and with proper indentation.
Comment 4 Hugues Fruchet 2016-01-21 10:48:33 UTC
Many thanks Tim-Philipp !

exclude list was pushed in order to be in sync with playbin way of doing, not used anyway on my side, so no problem on this.

Best regards,
Hugues.