nlecomposition: Fix small remaining race in previous commit

The seek action might currently be handled (in which case it is not in
the actions list and the action lock is not locked), but not actually
handled completely yet (the seqnum is not stored yet).

To prevent this, we remember what the current action is that is being
handled, and also compare to that.

https://bugzilla.gnome.org/show_bug.cgi?id=774149

tests_: Use nose2 instead of nosetests

Differential Revision: https://phabricator.freedesktop.org/D1394

ges: Fix documentation and debug comments

Reviewed-by: Thibault Saunier <thibault.saunier@collabora.com>
Differential Revision: https://phabricator.freedesktop.org/D1393

timeline: reimplement snap_to_position a bit more appropriately.

It could yet be made be simpler, but it would require
touching the rest of the timeline editing code.

Fixes https://phabricator.freedesktop.org/T7587

Reviewed-by: Thibault Saunier <thibault.saunier@collabora.com>
Differential Revision: https://phabricator.freedesktop.org/D657

nlecomposition: De-duplicate seek events based on their sequence number

If there are e.g. multiple video sinks, we would get the same seek event
multiple times. But we only want to handle it once.

https://bugzilla.gnome.org/show_bug.cgi?id=774149

test:validate: Port to python3

ges-timeline: Fix typo in debug messages

timeline: Avoid creating extra transition when rippling clips

In some cases when rippling clip we could get the algo lost because
a transition existed between two clips (for example at the end of c1
and at the begining of c2) but while rippling it would have required
a transition at the end of c2 and beginning of c1, and we were properly
not destroying the old one (as the two clips were in the moving context)
but we were still creating the other transition in the end...

Reviewed-by: Alex Băluț <alexandru.balut@gmail.com>
Differential Revision: https://phabricator.freedesktop.org/D1362

timeline: Make sure transitions between rippled clips are never deleted

Reviewed-by: Alex Băluț <alexandru.balut@gmail.com>
Differential Revision: https://phabricator.freedesktop.org/D1361

timeline: Destroy transition if a neighbor is not being moved to a layer

And make sure that we move the transition to the right layer, not trying
to figure it out.

Differential Revision: https://phabricator.freedesktop.org/D1360

tests_: Check transition is gone when editing clip to another layer

Differential Revision: https://phabricator.freedesktop.org/D1359

meson: Unset the plugin paths to generate the .gir files

Avoiding problems when using subproject:

'Failed to load plugin something.so file too short'

meson: update version

Back to development

Release 1.10.0

meson: Don't depend on gstreamer-check-1.0 on windows

https://bugzilla.gnome.org/show_bug.cgi?id=773114

Revert "meson: move gstreamer-check-1.0 dependency to tests/check"

This reverts commit 5665c2bfc9cae531c6dd9a75766d06a4af25ab9a.

Does not actually work. See:
https://bugzilla.gnome.org/show_bug.cgi?id=773114#c31

meson: move gstreamer-check-1.0 dependency to tests/check

meson: mark gstreamer-check-1.0 as required: false

meson: Don't set c_std to gnu99

Use the default for each compiler on every platform instead. This
improves our compatibility with compilers that don't have gnu99 as
a c_std.

meson: Use environment object to setup test environment variables

Bump meson requirement to 0.35

track-element: Avoid dereferencing NULL pointer

We set TrackElement track type very early when creating effects
so it now uses that information to find TrackElement in clips
by track type.

Reviewed-by: Alex Băluț <alexandru.balut@gmail.com>
Differential Revision: https://phabricator.freedesktop.org/D1370

meson: Add python tests

meson: Setup pre commit hook and fix getpluginsdir for standalone case

meson: update version

Release 1.9.90

meson: Fix gtkdoc using new meson features

meson: Fix installing configured files

uriclip: Remove some filesource leftovers

Differential Revision: https://phabricator.freedesktop.org/D1329

tests_:python: Factor out common code

Differential Revision: https://phabricator.freedesktop.org/D1328

nle: Drop tags getting out of the composition

Those tag are meaningless in for the new stream created by the composition

First step toward fixing T3070

Differential Revision: https://phabricator.freedesktop.org/D1327

timeline: Properly compute the end of groups when checking snapping

Computation was not taking into account the fact that the start of
the element being moved could be at the middle of a group and not
necessarily at the start!

Fixes T7544

Reviewed-by: Alex Băluț <alexandru.balut@gmail.com>
Differential Revision: https://phabricator.freedesktop.org/D1282

ges: Handle moving groups with effects inside

We were only concidering that we should let the group handle moving
transitions when changing transitions but in fact as soon as a
transition is happenning between two clips that are in a same group
the group properly handles moving the transition, so let the
group do its job.

Fixes T7543

Differential Revision: https://phabricator.freedesktop.org/D1281

ges: Deprecate GESTimelineElement::priority writability

GESLayer is now responsible for setting clips priorites. Also
GESClip top effects priorities are now set by the
ges_clip_set_top_effect_index method, the user should never call
ges_timeline_element_set_priority as it will anyway be overriden
by GES itself.

Differential Revision: https://phabricator.freedesktop.org/D1280

layer: Handle operation priorities

All operations should have higher priorites and sources should be
on top of those. We now first set the operations priorities in
a first pass and then stack sources on top of those.

Differential Revision: https://phabricator.freedesktop.org/D1279

ges: transition: Make crossfade fade out at the same time as it fade in

Until now fade out was just fading in the new clip, but this is not
correct and crossfade should at the same time fade out while fading
in.

Fixes https://phabricator.freedesktop.org/T3451

Differential Revision: https://phabricator.freedesktop.org/D1278

layer: Make sure to resync priorities on commit

In case effects have been added priorites might become wrong,
but until the timeline is not commited, it does not matter.

Make sure all priorities are correct before commiting compositions

Differential Revision: https://phabricator.freedesktop.org/D1277

Finally move clip priority handling to GESLayer.

Fix all tests as we now have 1 priority inside the layer
dedicated to transitions (basically no source clip will
ever have a priority of 0 inside a layer).

Differential Revision: https://phabricator.freedesktop.org/D1276

clip: Make top effect priority inside the clip priority range

And simplify the way we start computing children priority
making min_priority already relative to the clip itself.

Differential Revision: https://phabricator.freedesktop.org/D1275

tests_: timelineedition: Fix test now that we ripple from start and not from end

Differential Revision: https://phabricator.freedesktop.org/D1274

nlecomposition: Don't try to seek on an empty stack

We would seek on a NULL pad then, which gives ugly assertions.

https://bugzilla.gnome.org/show_bug.cgi?id=771843

nleurisource: Always provide a srcpad

By putting uridecodebin into a bin with a ghostpad. Without this,
nlesource tries to get a srcpad too early (before uridecodebin added
one) and everything fails miserably.

This has to be fixed properly in nlesource at some point, by properly
handling dynamically added pads. Currently they can only work if they
are added in states <= READY, which is not the usual case.

https://bugzilla.gnome.org/show_bug.cgi?id=771843

nlesource: Fail prepare() if no valid source pad is found

https://bugzilla.gnome.org/show_bug.cgi?id=771792

ges-launch: Be a bit more agressive sanitizing arguments

Otherwise GstStructure might fail parsing some fields
containing brackets

https://bugzilla.gnome.org/show_bug.cgi?id=771434

configure: Depend on gstreamer 1.9.2.1

timeline: Make get_groups public

Had to separate timeline_emit_group_added from timeline_add_group
to avoid emitting group-added when the project is being loaded.

Reviewed-by: Thibault Saunier <thibault.saunier@collabora.com>
Differential Revision: https://phabricator.freedesktop.org/D1302

tests_: Make sure child-removed is emitted when ungrouping

Reviewed-by: Thibault Saunier <thibault.saunier@collabora.com>
Differential Revision: https://phabricator.freedesktop.org/D1301

timeline: Fix documentation

Reviewed-by: Thibault Saunier <thibault.saunier@collabora.com>
Differential Revision: https://phabricator.freedesktop.org/D1300

Remove obsolete dependency

Reviewed-by: Thibault Saunier <thibault.saunier@collabora.com>
Differential Revision: https://phabricator.freedesktop.org/D1299

Automatic update of common submodule

From b18d820 to f980fd9

Automatic update of common submodule

From f49c55e to b18d820

tests: Move -DGES_TEST_FILES_PATH to common_cflags

As it is needed to build the utils lib

tests: Fix the way we get tests assets

Using __FILE__ won't work properly with meson.

test:validate: Handle new expected_failures Test argument

meson: Do not build libges against libcheck

meson: Properly find where other GStreamer plugins are when using subprojects

Revert "validate: Blacklist racy tests"

This reverts commit ce35412ff260fbd6e07b374bc3ca677053c277e0.

https://bugzilla.gnome.org/show_bug.cgi?id=769894 has been fixed

meson: Handle building the gir file when used as subproject

Add support for building GIR when used as subproject
Add an option to disable GIR generation
And bump version to 1.9.2

Back to development

Release 1.9.2

meson: doc: Fix building documentation when using subprojects

And check the presence of gtk-doc before building the documentation

ges: Rename parse_lex.h to ges-parse-lex.h

Fixes the build and makes it consistent with the meson build system.

Add support for Meson as alternative/parallel build system

https://github.com/mesonbuild/meson

ges-track-element: fix typos in control_binding_removed signal declaration

https://bugzilla.gnome.org/show_bug.cgi?id=770101

validate: Blacklist racy tests

See https://bugzilla.gnome.org/show_bug.cgi?id=769894

ges-timeline: Demote some debugging statements

locking should be in a lower level to avoid too many messages

timeline-element: Reset pointer after freeing

dispose can be called multiple times, make sure we don't call functions
on free'd pointers.

ges: Do not rescale videos if the track aspect ratio changes

Differential Revision: https://phabricator.freedesktop.org/D1242

ges: Let the compositor do the scaling if mixing is enabled

Differential Revision: https://phabricator.freedesktop.org/D1241

tests:validate: Also test opus and theora in OGG

validate: Start also testing jpeg encoding

timeline: Keep transitions when moving the moving context between layers

Differential Revision: https://phabricator.freedesktop.org/D1225

ges-launcher: don't leak project uri string

ges_project_get_uri returns a cloned string so it should
be free'd after usage.

Reviewed-by: Thibault Saunier <thibault.saunier@collabora.com>
Reviewed-by: Thibault Saunier <thibault.saunier@collabora.com>
Differential Revision: https://phabricator.freedesktop.org/D381

timeline: Ripple from start of clips and not the end.

Fixes https://phabricator.freedesktop.org/T7503

enums: Fix absolute text overlay alignment value

It needs to be in sync with GstBaseTextOverlayHAlign order.

title-source: Properly implement GESTimelineElement->lookup_child

GESTrackElement->lookup_child is deprecated and should be avoided
as much as possible.

clip: Reimplement look_child and iterate over children if needed

Otherwise in the case where children reimplement lookup_child to
handle some property renaming lookup fails.

tools: Fix printing commands help

ges:validate: Check that no extra decoding happens

uri-sources: Make sure to set decodebin 'caps' property

Fixes a regression where we decode streams twice,
this was introduced when we started creating NLE
object at GESTrackElement construct time.

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=769193

timeline: Emit snap-ended on commit if needed

Commiting the timeline means that the current operations on the clips are over,
so we should concider snapping as done at that point

Fixes T7499

fixed ges-version.h not found issue during out of tree build

image-source: Do not concider inpoints

We have no restriction on inpoint for Images

Differential Revision: https://phabricator.freedesktop.org/D1202

tests:validate: Better choose tested rendering formats

Automatic update of common submodule

From f363b32 to f49c55e

Back to development

Release 1.9.1

ges-launch: unref GApplication properly

Differential Revision: https://phabricator.freedesktop.org/D380

project_: improve get_uri doc

The return value of ges_project_get_uri should be freed
after usage.
Differential Revision: https://phabricator.freedesktop.org/D1142

Automatic update of common submodule

From ac2f647 to f363b32

tests: Make sure to run python tests against the build in tree

ges: Don't remove track elements from clips when removing from layer

And reuse the same previously created element when adding the clip
back to a layer, avoiding losing all setting done on clip children
in that situation

This is a behaviour change but previous behaviour was actually totally
unexpected and people working around that weird behaviour will moste
probably not care about that change

Differential Revision: https://phabricator.freedesktop.org/D1094

title_: Do not forget to link up child_added/removed vmethod

Otherwise effect handling is broken

Differential Revision: https://phabricator.freedesktop.org/D1099

auto-transitions: Do not remove auto transitions when moving neighboor from the same group

Differential Revision: https://phabricator.freedesktop.org/D1097

ges: fix various leaks with usage of ges_timeline_element_lookup_child

Some callers forgot to unref out child, pspec or both leading to leaks.

https://bugzilla.gnome.org/show_bug.cgi?id=766449

video-track: don't leak restriction caps in _sync_capsfilter_with_track()

https://bugzilla.gnome.org/show_bug.cgi?id=766450

ges-uri-asset: GstDiscoverer can return a valid info but a non-OK result, consider this an error

The asynchronous case in ges_uri_clip_asset_request_async() already considered
it an error, do the same in ges_uri_clip_asset_request_sync().

https://bugzilla.gnome.org/show_bug.cgi?id=767293

nlecomposition: Fix race condition in seek handling causing deadlocks

We might receive another seek from the application while the action task is
handling a previous seek (and thus setting seeking_itself to TRUE). To prevent
this seek to go through directly instead of being added as an action, also
check if the seek event was received from our action task thread or some other
thread.

https://bugzilla.gnome.org/show_bug.cgi?id=767053

ges: Emit GESMetontainer::notify-meta even if value is unset

ges: Allow passing `NULL` as a value to ges_meta_container_set_meta

Fixes T7430