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 564421 - Move appsrc/appsink to -base
Move appsrc/appsink to -base
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
git master
Other Linux
: Normal blocker
: 0.10.22
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on: 564482
Blocks:
 
 
Reported: 2008-12-13 21:29 UTC by David Schleef
Modified: 2009-01-06 11:09 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description David Schleef 2008-12-13 21:29:11 UTC
Let's move appsrc/appsink to -base before the (impending) freeze.
Comment 1 Sebastian Dröge (slomo) 2008-12-14 06:47:17 UTC
IIRC Wim wanted to support a few more use cases before getting it moved to -base (maybe they require an API breakage). If this is all done already it's a good idea to get it moved ;)
Comment 2 Wim Taymans 2008-12-15 12:03:49 UTC
I think I still want for appsrc:

 - latency configuration
 - capture delay (may or may not be in basesrc)

for appsink:

 - render delay 

It's new API so it can be added later too.
Comment 3 Wim Taymans 2008-12-15 13:53:20 UTC
        * libs/gst/base/gstbasesink.c: (gst_base_sink_class_init),
        (gst_base_sink_init), (gst_base_sink_set_property),
        (gst_base_sink_get_property):
        Expose the render-delay as a property so things like appsink can use it
        to tweak the synchronisation.
Comment 4 David Schleef 2008-12-15 23:07:42 UTC
Wim, so does that mean we should move it now, or wait?
Comment 5 Wim Taymans 2008-12-16 07:27:10 UTC
I'm still working on the latency configuration, will commit those later.
Comment 6 Wim Taymans 2008-12-16 16:05:02 UTC
It seems I forgot to commit this so it'll have to wait for when I'm back in Barcelona.

Anyway, this should not block a move, it's only a simple addition.
Comment 7 Wim Taymans 2008-12-29 16:46:01 UTC
        * gst-libs/gst/app/gstappsrc.c: (gst_app_src_class_init),
        (gst_app_src_init), (gst_app_src_set_property),
        (gst_app_src_get_property), (gst_app_src_query),
        (gst_app_src_set_latencies), (gst_app_src_set_latency),
        (gst_app_src_get_latency), (gst_app_src_push_buffer_full):
        * gst-libs/gst/app/gstappsrc.h:
        Add properties and methods to configure and retrieve the min and max
        latencies.
Comment 8 Jan Schmidt 2009-01-06 11:09:33 UTC
Appsrc/Appsink moved....