tests: Properly setup GST_PLUGIN_PATH in test environement

Dist gst-editing-services.doap

asset: Add a method to retrieve the GError of an asset loaded with error

API:
    ges_asset_get_error

project: Add a 'asset-loading' signal

ges: Set restriction caps in the audio source caps filter

Otherwise we could have not negotiated errors in audiomixer when
the channel/channel-mask do not match

Differential Revision: https://phabricator.freedesktop.org/D493
Reviewed-by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

formatter: Do not serialize top effect priorities

We just need to make sure they are always serialized in the right
order (which is the case) and de serializing them will lead to the
right behaviour.

We should not serialize the priority as the priority of the source
itself depends on the action having been done on the parent clip,
and we do not serialize the source priorities (and should not, GES
should just do the right thing).

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

launcher: Make sure to not activate validate twice when simply loading a scenario

uri-clip: Make sure to instantiate an asset to back GESUriClip-s

Automatic update of common submodule

From b99800a to b319909

Use new GST_ENABLE_EXTRA_CHECKS #define

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

Automatic update of common submodule

From 9aed1d7 to b99800a

Update GLib dependency to 2.40.0

nlecomposition: free closure actions when disposing

Summary:
After invoking GClosure, the item of action list becomes
orphan so it lost a chance to be freed. In addition, even
when disposing, the list of actions has few items so we
have to check the list.

Reviewers: thiblahute

Projects: #gstreamer_editing_services

Reviewed By: thiblahute

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

nlecomposition: fix wrong argument order of GClosureNotify

Summary:
_free_action should follow GClosureNotify type.

```
void
(*GClosureNotify) (gpointer data,
                   GClosure *closure);
```

Reviewers: thiblahute

Projects: #gstreamer_editing_services

Reviewed By: thiblahute

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

track: mixing_operation is handled by its parent

Summary:
Normally, mixing_operation is created and added to nlecomposition
as a child element so it will be freed when nlecomposition is removed
from a track.

Reviewers: thiblahute

Projects: #gstreamer_editing_services

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

nleoperation: don't leak iterator

Summary: Once an iterator is created, it should be freed after usage.

Reviewers: thiblahute

Projects: #gstreamer_editing_services

Reviewed By: thiblahute

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

asset: simplify if-statement in cache_set_loaded

Summary:
Manual iteration can be replaced with foreach function.
In addition, this patch fixes mismatched GFunc type for
g_list_foreach and adds debug cateory for gst-asset for
convenient debugging.

Reviewers: thiblahute

Reviewed By: thiblahute

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

uri-asset: do not reuse a passed GError pointer

Summary: A passed GError is re-allocated when discoverer has no information.

Reviewers: thiblahute

Projects: #gstreamer_editing_services

Reviewed By: thiblahute

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

xml-formatter: handle dispose properly

Summary:
To dispose properly, a child object should call same function
of parent class.

Reviewers: thiblahute

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

base-xml-formatter: properly handle GFile from wrong uri

Summary:
g_file_new_for_uri never fails so GFile always has valid pointer.
And fix a bug of double unref from D303.

Reviewers: thiblahute

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

don't leaks caps and converted strings

Summary:
Valgrind reports trivial leakages related to handling
objects and their converted strings.

Reviewers: thiblahute

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

track: add gaps when going from READY to PAUSED.

Summary:
The backend commits itself automatically in these cases, so track
needs to do so too.

Reviewers: thiblahute

Reviewed By: thiblahute

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

editing-services: Fix memory leaks when context parse fails

When g_option_context_parse fails, context and error variables are not getting free'd
which results in memory leaks. Free'ing the same.

And replacing g_error_free with g_clear_error, which checks if the error being passed
is not NULL and sets the variable to NULL on free'ing.

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

nleoperation: don't leak srcpad

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

project: fix a pointer for error message

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

project: don't leak GFileInfo

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

timeline-element,track,framepositionner: don't leak internal object

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

structured-interface: introduce TRY_GET_STRING

TRY_GET uses gst_structure_get. However, if boxed or
string pointer is retrieved by gst_structure_get,
it should be freed properly.

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

uri-asset: don't leak uri string

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

Revert "title-source: Force format with alpha channels out of videotestsrc"

This reverts commit 7d1e1010728a5348674bb9053de6b095cb824984.

This commit was never meant to be committed (at least *not* on master).

structure-parser: define GES_STRUCTURE_PARSER macro

And fix trivial leakages of internal list structure.

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

title-source: Force format with alpha channels out of videotestsrc

Making sure the user can set a background of the title with an alpha
channel.

Working around https://bugzilla.gnome.org/show_bug.cgi?id=755482 for
the 1.6 branch.

Back to development

Release 1.6.0

ges: Avoid emitting 'child-added/removed' when signal emission stops addition

In the GESTimeline, TrackElement addition to a clip might get cancelled
(and thus the element gets removed), we need to make sure users do not
get wrong signals.

Also document the fact that user should connect to container::child-added
with g_signal_connect_after.

nle{composition,object}: remove unused allocation & trivial leakages

nlecomposition allocates unused 'UpdateCompositionData' and it
causes leakages.

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

pipeline: don't leak GstPad

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

pitivi-formatter: don't leak internal hash table

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

video-transition: Make compositor background transparent

Allowing further mixing downstream

video-transition: Add a framepositioner at the end of the transitio

So downstream compositor knows the zorder of the various streams

Release 1.5.91

ges: Namespace NLE utils function into GES

Avoiding name clashes when built statically

nle: Avoid unsetting srcpad target after the srcpad is already freed

That leaded to segfaults

nle: Stop wrongly set operation segment base time

Inside the composition we actually do not need to have any notion
of what the timing outside the compositon as we already tweak the segment
base time outside the composition. This code was only there to work
around https://bugzilla.gnome.org/show_bug.cgi?id=753196

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

nlecomposition: don't leak internal hashtable

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

nleobject: don't leak srcpad when disposing

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

timeline: don't leak pad in private structure

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

ges-launcher: don't leak GError

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

ges-launcher: fix double free when argument is invalid

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

video-source: Use the priority being set to compute zorder

video:transition: Set mixer pad zorder

video-source: Make sure to set framepositionner zorder when creating it

And fix a computation bug where we would be having mixing order
reversed between layers.

And make sure that the positionner does not mix up Transition handling
of the zorder

video-source: Simply set framepositionner->zorder = self->priority

Summary:
Making the code simpler and handling the transition case
where elements are in the same layer (which was failing
/setting same zorders until now).

Reviewers: Mathieu_Du

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

Use standard GST_PLUGIN_LDFLAGS for the nle plugin

Add the standard GST_PLUGIN_LDFLAGS to the configure.ac file.

Add support for static plugins builds

Release 1.5.90

Move NLE to a dedicated GstPlugin

Summary: Allowing external user to directly use it

Reviewers: Mathieu_Du

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

nle: Enhance debug logging

ges: Do not leak and uselessly create errors

And avoid parenthesis in GstObject names

ges: Fix how we handle layer vs layer-priority in the structured interface

tests: Do not use gst-structured-interface in the tests

It breaks $ make distcheck

Update mailing list in doap file

timeline: Remove transitions that can no fit into an auto transition

When activating auto transition mode

ges; Minor debug enhancement

assets: Avoid deadlock when done initialising asset

Avoid to hold the CACHE lock when setting the GTasks return values.

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

nleobject: Concider objects as 'inactive' when they have a duration == 0

track-element: Handle the case where we have only one keyframe set when interpollating keyframes

asset: Port use of deprecated GSimpleAsyncResult to GTask

Automatic update of common submodule

From f74b2df to 9aed1d7

track-element: Fix splitting bindings and add unit tests

structured-interface: Better handle CLOCK_TIME type from GstStructures

element: Implement a paste method

Allowing user to copy paste clips very easily

timeline: Disable movement that lead to 2 transition at a position

We should never let 3 objects to overlap at a same position, for that
we introduce a "rollback" feature and whenever such an editing happens,
we rollback object position to whatever it was before the move.

smart-video-mixer: Always keep a ref on the mixer pad

timeline-element: Add a method to get the TrackType it interacts with

API:
  + ges_timeline_element_get_track_types

timeline: Never create transitions between rippled objects

In case of groups, we can have track elements that do not belong
directly to the moved_trackelements but will be moved as others. Never
create transition to all object that have a start > moving group start.

video-transition: Use a SmartMixer as mixer

So that the frame position metas are parsed and taken into account

track-element: Add method to remove control binding

API:
  ges_track_element_remove_control_binding

ges: Handle absolute GstDirectControlBindings

framepositionner: Make use of the new CompositorPad.width/height

So that the scaling is done in the compositor and this way we can cleanly interpolate its value

timeline: Never change output media time when trimming start

+ Fix testsuite

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

track-element: Return right value when editing

We used to always return TRUE which was wrong

+ Fix testsuite and remove randomness from the tests

ges: Do not add a final gap at the end of track while rendering

It is not correct to force a black frame at the end of the rendered
video and it also leads to rendering issue with vpX encoders.

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

clip: Use container priority offset when setting children prios

Instead of trying to compute it ourself which might lead to wrong
behaviour when moving between layer.

+ Make sure that when we reset clip children priority (to make space
  for effects,) we update the container knowledge of priority offsets

clip: Fix track element priority computation

We were computing the priority offset taking the global MIN_NLE_PRIO
(which is a constant == 2 to make space for the mixing elements) instead
of the layer 'track element' relative priority, leading to very big
offsets on layer with a prio > 0. In the end it leaded to effects having
the same priority as the sources which leads to an undefined behaviour
in NLE.

uri-asset: Bump the discoverer timeout to 1 minute

We should by default avoid false timeouts

track: Give usefull name to compositions

nle: Port tests to the "commit" action signals

Now that nle_object_commit symbol is hidden, we can't use it
in the tests.

ges: Unbreeak API after renaming of GNL to NLE

ges: Hide more symbols of headers that are not installed

nle: Remove unused function

nle/nlecomposition.c:2471:1: error: unused function '_parent_or_priority_changed' [-Werror,-Wunused-function]
_parent_or_priority_changed (NleObject * obj, GNode * oldnode,
^

nle: Hide away symbols, they're supposed to be internal

Back to development

Release 1.5.2

nlesource: remove outdated comment

Automatic update of common submodule

From 6015d26 to f74b2df

pipeline: Handle rendering with disabled tracks

Summary:
The user might want to render only some media type of the timeline,
for example he wants to only render the audio part of the timeline.

It was failing as we were not connecting the track but were still trying
to 'render' it.

Depends on D153

Reviewers: Mathieu_Du

Reviewed By: Mathieu_Du

Differential Revision: http://phabricator.freedesktop.org/D154

ges: Raise an error when the discoverer returns != RESULT_OK

And do not try to run the pipeline when that happens

uri-asset: Add a way to control discoverer timeout through envvar

Making it possible to run ges-launch test under valgrind for example

ges-launch: don't print random position/duration values at startup