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 751367 - dvbbasebin: optional support of target device in uri
dvbbasebin: optional support of target device in uri
Status: RESOLVED OBSOLETE
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
1.4.5
Other Linux
: Normal enhancement
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2015-06-23 09:04 UTC by Hugues Fruchet
Modified: 2018-11-03 13:36 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
dvbbasebin: optional support of target device in uri (5.54 KB, patch)
2015-06-23 09:04 UTC, Hugues Fruchet
none Details | Review

Description Hugues Fruchet 2015-06-23 09:04:04 UTC
Created attachment 305901 [details] [review]
dvbbasebin: optional support of target device in uri

The attached patch improves dvb uri handler to let user set the target DVB device. This is usefull when several DVB frontend cards are connected on the same device.

Target device is given in uri using syntax "dvb://<device>/<channel name>":

 gst-launch-1.0 playbin uri="dvb:///dev/dvb/adapter0/frontend1/France 2"

This was tested on B2120 ST reference board running STiH412 'Monaco' chipset with 2x DVB-T frontend card connected.

Tested on GStreamer-1.4.5, but patch attached is rebased on master branch.
Comment 1 Hugues Fruchet 2015-09-09 08:58:16 UTC
Could this patch be reviewed ? Any chance to reach GStreamer-1.6 ?
Comment 2 Tim-Philipp Müller 2015-09-09 13:15:11 UTC
Sorry this fell through the cracks. Git master is frozen now for new features, and this patch is not exactly a one-liner :) I'd say ping again once 1.6 is out then we can get it in once git master is open for new features again.
Comment 3 Hugues Fruchet 2015-09-09 13:28:30 UTC
Ok thks for answer, I'll ping again later.
Comment 4 Hugues Fruchet 2015-11-02 12:44:53 UTC
Hi Tim-Philipp,

Could someone get a look to this patch ?

BR,
Hugues.
Comment 5 Wonchul Lee 2016-05-09 03:42:41 UTC
It seems to be intended to accept URI from the DVB source not only dvb://<channel> but also dvb://<dev>/<channel> with the adapter and frontend values.
I think It will be good to set those value through the URI (though the adapter could have been set on env variable). This could be the another option to provide adapter/frontend value to the source.

This patch could bring some problem to someone else who using dvbbasesrc with the URI consisted of dvb://<channel> and channels.conf. The set_properties_for_channel search the specific properties based on channel_name but it mixed with "/dev/adapter%/frontend%/channel_name".

So, how about parsing adapter/frontend value from the URI before set_properties_for_channel if exist, and pass the correct channel_name, then set the adapter/frontend properties to dvbsrc.
This makes it more reliable for the other use cases.
Comment 6 GStreamer system administrator 2018-11-03 13:36:30 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/issues/262.