Release 1.11.90

completion: Fix previous commit

completion: Fix listing commands

Executing a single string does not work in this context. Fixed using
a bash function instead.

completion: Remove incorrect fixmes

In fact the fixmes are incorrect since these are options that are
available when built against gst-validate, which I didn't. There is
reference to these options in the HELP_SUMMARY that refers to these
options stating "if ges-launch is built with gst-validate ..." and these
get picked by the regex that list the options.

completion: Update to new gstreamer core helpers

Also fix regressions, and mark lost features and problems with fixmes.

Tiny fixes in the documentation

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

Add missing win32 definition

This should fix dist check. ges_layer_get_clips_in_interval() was
added recently but missing from the list.

meson: Use get_pkgconfig_variable instead of calling pkg-config ourself

It is avalaible in meson 0.36 which is now are requirement

Nothing happens on not found dependencies.

meson: test: Fix environment object usage

And make sure to bring -good plugins in.

layer: Add ability to get clips in a given interval

Reviewed-by: Alex Băluț <<alexandru.balut@gmail.com>>
Reviewed-by: Thibault Saunier <thibault.saunier@collabora.com>
Differential Revision: https://phabricator.freedesktop.org/D1689

docs: Fix generation using markdown for titles around tables

Revert "doc: Build documentation with hotdoc"

This reverts commit 8857e004f78ea009e1c87a93da5cf3e25dbde07f.

This was not meant to be pushed yet.

Revert "Fhotdoc"

This reverts commit 220618ecc7c061a2146e00e0063123b8dbaeb734.

This was not meant to be pushed.

Revert "HOTDOC"

This reverts commit 5e251483ee6777b6a74a7988b5969bf95f6ecab6.

This was not meant to be pushed.

ges: Minor GESTimeline documentation fix

HOTDOC
gi

extractable: Typedef only in its own .h

Fhotdoc

docs: Port all docstring to gtk-doc markdown

doc: Build documentation with hotdoc

timeline: Return FALSE when commiting an empty timeline

Meaning that ASYNC_DONE/COMMITED is always emited when TRUE is returned

framepositioner: Prevent division by zero

CID 1369046
CID 1369047

ges-timeline: Document intentional case-fall-through

CID 1364754

ges-ui: Ensure that string is \0-terminated

CID 1320699

ges-project: Check for set/unset error correctly by dereferencing

... or simply calling g_clear_error() on it which does that for us.

CID 1257630

examples/ges-ui: Remove useless NULL check

g_new0() will abort if allocation fails.

CID 1139842

ges: Keep internal symbols internal

The following implementation details where exposed as public symbols:
  - _ges_container_get_priority_offset
  - _ges_container_set_height
  - _ges_container_set_priority_offset
  - _ges_uri_asset_cleanup

but it was not correct and that should never have been used outside
GES.

Moving those declarations to the internal header and marking as
internal.

meson: Update version

Back to development

Release 1.11.2

meson: Add an option to disable doc generation

Minor documentation fix

meson: dist meson build files

Ship meson build files in tarballs, so people who use tarballs
in their builds can start playing with meson already.

win32: Update def file

mixers: fix leaks in tests

- GstMessage and GstBus references were lost
- Need to call gst_bus_remove_signal_watch() for each
gst_bus_add_signal_watch_full() call

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

ges-meta: Minor documenation fix

introduce ges_deinit()

GstDiscoverer objects were leaked by tests making the leaks detector
unusable.
Introduce ges_deinit(), similiar to gst_deinit(), doing some cleanup
before exiting the process.

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

uri-clip-asset: Add file-size metadata

Add file-size metadata to GESUriClipAsset.

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

timeline: Cleanup the moved_clip list before rolling back

Otherwise we might end up using an already freed pointer

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

Revert "meson: don't use subproject fallback for gst-validate if it won't work"

This reverts commit 6760e5e0b1b2f28fb04e9c430506af56c15432b9.

This was not supposed to be pushed and should not be needed any more.

meson: bump version

meson: don't use subproject fallback for gst-validate if it won't work

gst-validate has a hard-dep on json-glib-1.0 so maintain optionality of
it all by only dragging it in as a fallback if we know we can satisfy
the dependencies.

Back to development

Release 1.11.1

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