nlesource: Protect seeks from tear down

Otherwise there is a race where we trigger the seek at the exact
same time the composition is being teared down potentially leading
to basesrc restarting its srcpad task which ends up being leaked.

Fixes ges.playback.scrub_backward_seeking.test_title.audio_video.vorbis_theora_ogg
and probably all its friends timeouting with the following stack trace:

     (gdb) t a a bt

     Thread 4 (Thread 0x7f5962acd700 (LWP 19997)):
     #0  0x00007f5976713efd in syscall () at ../sysdeps/unix/sysv/linux/x86_64/syscall.S:38
     #1  0x00007f5976a9d3f3 in g_cond_wait (cond=cond@entry=0x7f5938125410, mutex=mutex@entry=0x7f59381253c8) at gthread-posix.c:1402
     #2  0x00007f5976c9e26b in gst_task_func (task=0x7f59381253b0 [GstTask]) at ../subprojects/gstreamer/gst/gsttask.c:313
     #3  0x00007f5976a7ecb3 in g_thread_pool_thread_proxy (data=<optimized out>) at gthreadpool.c:307
     #4  0x00007f5976a7e2aa in g_thread_proxy (data=0x7f5954071d40) at gthread.c:784
     #5  0x00007f59767ea58e in start_thread (arg=<optimized out>) at pthread_create.c:486
     #6  0x00007f59767196a3 in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:95

     Thread 3 (Thread 0x7f5963fff700 (LWP 19995)):
     #0  0x00007f597670e421 in __GI___poll (fds=0xe32da0, nfds=2, timeout=-1) at ../sysdeps/unix/sysv/linux/poll.c:29
     #1  0x00007f5976a553a6 in g_main_context_poll (priority=<optimized out>, n_fds=2, fds=0xe32da0, timeout=<optimized out>, context=0xe31ff0) at gmain.c:4221
     #2  0x00007f5976a553a6 in g_main_context_iterate (context=0xe31ff0, block=block@entry=1, dispatch=dispatch@entry=1, self=<optimized out>) at gmain.c:3915
     #3  0x00007f5976a55762 in g_main_loop_run (loop=0xe32130) at gmain.c:4116
     #4  0x00007f59768db10a in gdbus_shared_thread_func (user_data=0xe31fc0) at gdbusprivate.c:275
     #5  0x00007f5976a7e2aa in g_thread_proxy (data=0xe1b8a0) at gthread.c:784
     #6  0x00007f59767ea58e in start_thread (arg=<optimized out>) at pthread_create.c:486
     #7  0x00007f59767196a3 in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:95

     Thread 2 (Thread 0x7f5968dcc700 (LWP 19994)):
     #0  0x00007f597670e421 in __GI___poll (fds=0xe1bcc0, nfds=1, timeout=-1) at ../sysdeps/unix/sysv/linux/poll.c:29
     #1  0x00007f5976a553a6 in g_main_context_poll (priority=<optimized out>, n_fds=1, fds=0xe1bcc0, timeout=<optimized out>, context=0xe1b350) at gmain.c:4221
     #2  0x00007f5976a553a6 in g_main_context_iterate (context=context@entry=0xe1b350, block=block@entry=1, dispatch=dispatch@entry=1, self=<optimized out>) at gmain.c:3915
     #3  0x00007f5976a554d0 in g_main_context_iteration (context=0xe1b350, may_block=may_block@entry=1) at gmain.c:3981
     #4  0x00007f5976a55521 in glib_worker_main (data=<optimized out>) at gmain.c:5861
     #5  0x00007f5976a7e2aa in g_thread_proxy (data=0xe1b800) at gthread.c:784
     #6  0x00007f59767ea58e in start_thread (arg=<optimized out>) at pthread_create.c:486
     #7  0x00007f59767196a3 in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:95

     Thread 1 (Thread 0x7f5975df4fc0 (LWP 19993)):
     #0  0x00007f5976713efd in syscall () at ../sysdeps/unix/sysv/linux/x86_64/syscall.S:38
     #1  0x00007f5976a9d3f3 in g_cond_wait (cond=cond@entry=0xe34020, mutex=0xe39b80) at gthread-posix.c:1402
     #2  0x00007f5976a7f41c in g_thread_pool_free (pool=0xe34000, immediate=0, wait_=<optimized out>) at gthreadpool.c:776
     #3  0x00007f5976c9f1ca in default_cleanup (pool=0xe256b0 [GstTaskPool]) at ../subprojects/gstreamer/gst/gsttaskpool.c:89
     #4  0x00007f5976c9e32d in init_klass_pool (klass=<optimized out>) at ../subprojects/gstreamer/gst/gsttask.c:161
     #5  0x00007f5976c9e502 in gst_task_cleanup_all () at ../subprojects/gstreamer/gst/gsttask.c:381
     #6  0x00007f5976c214f4 in gst_deinit () at ../subprojects/gstreamer/gst/gst.c:1095
     #7  0x000000000040394f in main (argc=6, argv=<optimized out>) at ../subprojects/gst-editing-services/tools/ges-launch.c:94

meson: Build benchmarks

asset-uri: Create a specific discoverer when discovering sync

To allow 'reintrancy'.

This was a 'regression' introduced in bad64296d9b497a13f5f7fe91d568d85ed236265

Fixes https://gitlab.gnome.org/GNOME/pitivi/issues/2278

Mark ges_timeline_load_from_uri as deprecated

Update deprecation warning to match GTK style

Mark ges_formatter_save_to_uri as deprecated

tests: Add inconsistent init/deinit test case

ges: Enhance ges_{init/deinit} documentation

Add some init/deinit related comment and make assertion when
ges_deinit() is called from unexpected thread.

tests:python: Use proper GES.Project constructor

Avoiding a g_critical

nlecomposition: Suppress error from child during sync state with parent

This commit is to ensure cleanup internal elements on state change failure.

nlecomposition posts its own error message after cleanup child.
If we don't suppress child error, meanwhile, an application
triggered downward state change (resulting from child error message)
might be able to reach nlecomposition before internal cleaning child up.
That eventually results to downward state change failure.

nlesource: Don't leak pending seek event on dispose

nlecomposition: Don't keep sync state of child on activation failure

This will result in downward state change failure eventually
when user is finalizing top level (i.g., gespipeline) bin.

examples: add gessrc example binary to .gitignore

gesdemux: don't use deprecated gst_uri_construct()

Fixes #64

Back to development

ges: Link ges plugin to libgstpbutils

/usr/bin/ld: .libs/libgstges_la-gesdemux.o: in function `ges_timeline_new_from_uri_from_main_thread':
./plugins/ges/gesdemux.c:279: undefined reference to `gst_discoverer_new'
/usr/bin/ld: ./plugins/ges/gesdemux.c:288: undefined reference to `gst_discoverer_start'

Release 1.15.2

examples: add new gessrc example, so sourcefile gets disted

plugins: add autotools build for new ges plugin

examples: Add an example about using keyframes in python

ges-meta-container: Fix g-i annotation

ges-meta-container.c:516: Warning: GES: invalid "allow-none" annotation:
only valid for pointer types and out parameters

misc: Fix warnings on Cerbero's ancient MinGW

gesdemux.c:297:3: error: value computed is not used [-Werror=unused-value]

Fix segfault when adding clips to group outside a timeline

Making sure that objects are inside a timeline before adding/removing them from it

Fixes https://gitlab.freedesktop.org/gstreamer/gst-editing-services/issues/35

tests: Fix usage of undefined class

launcher: Add options to set tracks restriction caps

tests: ges: Fix various leak

layer: Fix asset leak

effect: Fix string leak

uri-asset: Implement dispose vmethod for GESUriSourceAsset

... and fix DiscovererStreamInfo leak

uri-asset: Don't forget to unref DiscovererInfo on dispose

Dispose() must unref DiscovererInfo ownership
taken by ges_uri_clip_asset_set_info().

video-transition: Fix GstPad leak

Returned Gstpad by link_element_to_mixer_with_smpte()
has increased refcount in ges_smart_mixer_get_mixer_pad().

timeline: Better handle loading inconsistent timelines

Auto transition when having 3 overlapping clips in a same point in the
timeline is not supported as we can't handle it in a nice way. Before we
to avoid creating 2 overlapping transitions (which is plain broken in
NLE) were completely disabling `auto-transition` and removing all
auto-transitions in the timeline but this is pretty weird for the end
user. This commit changes and now makes sure 2 transitions are not
created in the same place.

Also cleanup previous test case.

s/accured/occurred/g

xml-formatter: Minor debug enhancement

plugins: Add an a gesdemux element to 'demux' serialized timelines

plugins: implement a gessrc element useable from playbin

This is a new simple GstBin that can handle the ges:// uris
and will directly expose the srcppads of the tracks present in the
timeline.

uri-asset: Use the same code path for sync discovery as the async one

And start handling relocated assets.

Also expose the discoverer callback as a vmethod so that we can
overridde the discoverer when necessary (to handle discovering of
timeline through gesdemux for example)

nlecomposition: Get overall pipeline position by recursing up

And handle NLEComposition inside NLEComposition

Fixes https://gitlab.freedesktop.org/gstreamer/gst-editing-services/issues/39

videomixer: Drop allocation query after the compositor

Working around https://gitlab.freedesktop.org/gstreamer/gstreamer/issues/310

layer: Resort clips before syncing priorities

We set the priorities making the assumption that `start_clips` is properly
ordered by start!

Fixes https://gitlab.gnome.org/GNOME/pitivi/issues/2254

tests: asset: Add test async asset request with custom GMainContext

... and test call ges_{init/deinit} multiple times in a unit test.

Revert "ges: Add missing type unref on deinit"

This reverts commit e939cfebaf4deeabf21ba799ddc3eeaa87e7cf9a.

Class might not be initialized if they were already registered
when ges_init() was called, but were not created until ges_deinit() called.

ges: Add check sync/async discoverer

To support ges_{init/deinit} multiple times in a process,
there should be a method for setting up internal object/table of
GESUriClipAssetClass. because *_class_init() will be called
only once in process lifecycle.

uri-asset: Add missing GHashTable cleanup

... and use g_object_unref() for GFile object, it's not a GstObject.

ges: Print initialize error reasons

tests: asset: Remove out-of-date comment

tests: asset: Remove pointless gst_init()

It's done by GST_CHECK_MAIN() already

tests: init/deinit per test case

... in order to verify init/deinit pair.

ges: Simplify init/deinit flag

In theory, GES can be init/deinit multiple times in a process.
To simplify that use-case, let's trace only "ges_initialized" flag.

uri-asset: Add ges_uri_clip_asset_finish to get better introspection

Vala requires a matching _finish function to correctly bind the method with the right finish method.

timeline: fix two issues in the documentation

xml-formatter: Do not forget to serialize clips metadata

xml-formatter: Serialize groups metadatas

uri-asset: Don't leak GstDiscovererInfo

uri-asset: Impl. dispose vfunc

... and fix GList/GESAsset leak

transition-clip: Don't leak GESAsset

Returned GESAsset from ges_asset_request should be freed since
ges_extractable_set_asset doesn't take ownership

framepositioner: Fix invalid memory access

The GstFramePositioner might be finalized before the notify callback

Without this commit,
gst-editing-services / ges_basic / test_ges_timeline_remove_track
can reproduce the case.

nleoperation: Fix GstPad leak

Returned GstPad by nle_object_remove_ghost_pad() has increased
refcount.

nleghostpad: Fix GstEvent leak

nlecomposition: Don't leak GNode

Clear the last node before update

tests: nle: Fix various leak

Don't leak GError and GstPad object

tests: nle: Fix GList leak

tests: ges: Fix various leak

asset: Fix various leak

enums: Add missing unref

nlecomposition: Clear all members before chaining up to parent on finalize

structure-parser: Add missig chain up code

ges: Add missing type unref on deinit

ges: Cleanup internal hash table on deinit

System-wide once allocated but it makes tracing leak hard

ges: Make init/deinit thread safe

Although it might be uncommon use case, init/deinit could be called
in non-main thread.

asset: Use static lock

The mutex life cycle follows processs.

tests: Increase timeout value to 360 sec

Use consistent timeout value with core and other plugins.
Otherwise, valgrind sometimes timed out with default timeout 30sec.

nlecomposition: Don't try dump null stack

Fixes following assertion
Unexpected critical/warning: g_node_traverse: assertion 'root != NULL' failed

Release 1.15.1

ges: Register formatters during meta registration

So that formatters implemented outside GES itself are registered

ges: Add a ges_is_initialized function

formatter: sink ref of the temporary GESFormatter

To accomodate formatters implemented with bindings/in python

pipeline: Ensure timeline state to be NULL on dispose

The GESTimeline's state might not be synced with parent

nlecomposition: Handle state change failure

Whatever the reason for failure, try cleanup child elements
and internal thread.

ges-meta-container: Fix warning message

ges-meta-container: Minor documentation fixes

track-element: Ignore writability for whitlisted children props

If the property was explicitely whitelisted, we should expose it
in any case.

This was a regression from 835d69374978208bc73a8f823b899f624dda9479

ges: avoid use of G_DECLARE_FINAL_TYPE which requires GLib 2.44

Fixes https://gitlab.freedesktop.org/gstreamer/gst-editing-services/issues/48

container: Fix GHashTable leak

container: Fix wrong finalize() usage

finalize must chain up to parent's finalize(), not dispose()

timeline-element: Chain up to parent impl. on dispose()

... as documented in glib

WIP: ges: fix API export/import and 'inconsistent linkage' on MSVC

Export GES library API in headers when we're building the
library itself, otherwise import the API from the headers.

This fixes linker warnings on Windows when building with MSVC.

Fix up some missing config.h includes when building the lib which
is needed to get the export api define from config.h

Fixes https://gitlab.freedesktop.org/gstreamer/gst-editing-services/issues/42

build: also allow building static libraries for e.g. Android/iOS

Automatic update of common submodule

From ed78bee to 59cb678

Add a way to disable xptv support

This formatter is in very bad shape and is generally not useful.

It has been deprecated since 1.0... and I bet noone uses it.

Run gst-indent through the files

This is required before we enabled an indent test in the CI.

https://gitlab.freedesktop.org/gstreamer/gstreamer-project/issues/33

track: Fix documentation about "binding_type"

Revert "meson: Fix the reference to libxml2 path"

It seemed to be what the wrap file expected but in the end it is
just a bug in meson which is now fixed.

This reverts commit cc5d74d0be30dab92d1540ed749eaf4dcedd9171.

meson: Fix the reference to libxml2 path

meson: Add a fallback for libxml2

validate: cleanup the playback-time from validate structures

Otherwise we might fail on them in the ges-structure-interface

effect: Create ghost pads ourself

As we can have effects with several pads and the default ghosting
doesn't allow that.

This way we also filter the pads to ghost to match our track type.

effect: Consider the "Filter" classification to determine effect media type

Add Gitlab CI configuration

This commit adds a .gitlab-ci.yml file, which uses a feature
to fetch the config from a centralized repository. The intent is
to have all the gstreamer modules use the same configuration.

The configuration is currently hosted at the gst-ci repository
under the gitlab/ci_template.yml path.

Part of https://gitlab.freedesktop.org/gstreamer/gstreamer-project/issues/29

clip: Emit additional signals after child-removed

When removing an effect from a clip, first the notify::priority signals
were being emitted for the remaining effects which changed priority, and only
at the end the child-removed signal. Now the child-removed signal is emitted
first.