clip: Make sure that clip start change is notified before children changes

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

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

tests_: Check the order of signals when a transition is created

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

asset: Fix set_proxy to abort when an error happens

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

asset: Reuse local variable

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

meson: generate pkg-config -uninstalled pc files

Generating those files is useful for users building the GStreamer stack
using meson and having to link it to another project which is still
using the autotools.

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

ges-launch: Use standard GstEncodingProfile deserialization function

validate: Remove space breaking muting ges-launch

video-mixer: Fix the way we release mixer pads

We were using the actual mixer pad to release the smart mixer
pad, which seemed to be on purpose, but was not properly handle,
moreover, it is now forbiden to pass a pad not inside a GstElement
when releasing it.

Also properly remove ghost pads from Smart mixer, we were planly
failling at it.

meson: Install ges-launch

Remove generated .spec file

Likely extremely bitrotten, and we should not ship this anyway.

tests: Remove now meaningless empty testsuite

ges: add a basic unix man page for ges-launch-1.0

Do not list all the possible options in the man page but only the help
options.

This is in order to avoid duplication and prevent the man page from
becoming obsolete in case the options change in the code but do not get
updated in the man page.

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

ges: fix the description of the --help-GES command line option

Use "Show GES Options" which is more appropriate and avoids duplication
with --help-gst which already says "Show GStreamer Options".

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

tests: Make sure tests can be listed

Initializing GstCheck before creating the testsuite

ges-launch: Make sure GStreamer is always initialized

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

meson: Support building without Gst debug

Automatic update of common submodule

From 1416c16 to 39ac2f5

element: Rework set_child_property_by_pspec

It was making no sense to loose the information about the pspec itself
to retrieve the child associated to it and was failling when we were
forcing the AssociateType::prop synthax

test:nle: Soften check on refcount

The composition might already have taken a new ref processing the
source.

nlecomposition: Deactivate current stack in PAUSED_READY state

To avoid a race when tearing down the composition (PAUSED_TO_READY),
we should make sure to tear down the current stack and let the GstBin
class handle the remaining thing to do during the change state.

We should still ignore any error happening when tearing down the
bin state just in case.

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

tools: Deinit Gst before exiting

meson: Do not print error logs when building the gir

examples: remove #include <ges-internal.h> from assets.c

It is not needed and pulling it in is causing a link problem with msvc.
Including ges-internal.h sets the default debug category in assets.c to
_ges_debug. Because _ges_debug is marked as DATA in the libges.def, it
will only be linked from libges.dll if it is marked in the source with
dllimport. Instead of messing with that we can just remove this include.

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

common: use https protocol for common submodule

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

ges-timeline: Properly calculate absolute diff of two unsigned integers

CID 1394491.

ges: Add NULL check before dereferencing

CID 1394494.

ges-uri-asset: fix compile error 'timeout' may be used uninitialized

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

make: include common/win32.mak

With the addition of the .def file for libges we need to make
sure the check-export script from common gets executed so that the
.def stays up to date.

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

Update common submodule

Enable building with MSVC

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

Cast away const from GstMetaInfo in *_get_meta_info() functions

MSVC warns about the const in the implicit argument conversion in the
calls to g_once_init_{enter,leave}. It's OK so explicitly cast it.

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

Pass gint/guint pointers instead of enum pointers

The underlying integer type for enums are implementation defined and may
not be the same size as gint/guint. So implicitly casting from pointers-
to-enum-types to pointers-to-int-types is unsafe. MSVC warns on these.

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

parse: Don't #include <unistd.h>

It isn't needed and isn't present in non-posix environments like windows
with MSVC or mingw.

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

Revert "nlecomposition: Start task and initialize the stack after chaining up to parent's change state function"

This reverts commit 57d40bec1a3c5048baaad08403d7b7e641a9c55c.

Apparently it causes timeouts in the unit tests on Jenkins and
Thibault's machine, and in the gst-validate tests.

Caused by elements staying in PAUSED and waiting to be set to PLAYING.
Needs further investigation.

Revert "nleobject: Start up in NULL->READY->PAUSED after the parent class did"

This reverts commit 5f7943c59d9def8c2dc9983936463462c1cdf63f.

nleobject: Start up in NULL->READY->PAUSED after the parent class did

This keeps everything in a more consistent order and makes sure that the
base class is already set up completely before we start doing anything.
It also prevents from doing any setup if the base class fails, and
possibly not shutting things down again then.

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

nlecomposition: Start task and initialize the stack after chaining up to parent's change state function

Otherwise we could set the state of the children to PAUSED already (i.e.
start dataflow) from the composition's task, while the composition
itself is currently chaining up to the parent class' change state
function and did not activate the pads yet. This causes buffers and
events to be discarded, and everything to stop with a not-negotiated
error.

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

tests: Fix running python unit tests

Adding missing nose2-junit-xml.cfg.in file and minor fixes in
the way we call nose2 also making sure the .xunit files end
up in the right place.

ges: Check if GstDiscoverer could be created at init time

And fail initialization if it is not the case, we make the assumption
it worked all around the codebase so we should really concider it fatal.

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