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

Automatic update of common submodule

From d9a3353 to 6015d26

Automatic update of common submodule

From d37af32 to d9a3353

Automatic update of common submodule

From 21ba2e5 to d37af32

Automatic update of common submodule

From c408583 to 21ba2e5

Automatic update of common submodule

From d676993 to c408583

timeline: Never snap end when rippling

http://phabricator.freedesktop.org/T74

timeline: Never create transition between elements inside the moving context

http://phabricator.freedesktop.org/T74

group: Disconnect from old layer notify::priority when a clip is moved to a NULL layer

This means we need to properly track the layer a clip was in. We now
keep track of the various signal IDs in a dedicated structure and
keep a ref on the layer an object is in.

http://phabricator.freedesktop.org/T88

tools: Exit the app as it is a simgle instance app

And force exiting GstValidate when wanted

ges: remove dead code

Summary:
No need to recheck if error exists since it has already been checked by the
conditional above.

Coverity CID #1302832

Reviewers: thiblahute

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

ges: Handle trimming auto transitions

Meaning trimming neighbors.

+ And add a test

timeline: Use a simple GList to track auto transitions

nlecomposition: Do not fail when removing/adding child without commiting

Summary:
We use to end up removing the nleobject when the following case happened:

 * add an object
 * remove that object
 * re add the object
 * commit the composition

Reviewers: Mathieu_Du

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

timeline: Minor documentation addition

ges: Enhance xges format versioning

Summary:
Handle the fact that some new features can be added and that means
generated files will not be fully understandable by older versions of
the formatter.

Make sure that we set the format version to 0.2 when we serialize the
GstEncodingProfile.enabled property.

Add some tests around that.

+ Fix a minor bug in the test-utils
+ Add a meta on the projects to tell in what format version a project
  has been serialized/parsed back

API:
  GES_META_FORMAT_VERSION

Depends on D178

Reviewers: Mathieu_Du

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

ges: If last added clip is not in a layer, get the first layer

Summary:
In case we just removed it from its layer, make sure to
just use the first layer when none specified.

Depends on D177

Reviewers: Mathieu_Du

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

ges: Fix some error settings

Summary: Depends on D176

Reviewers: Mathieu_Du

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

ges:structured-interface: Use GET_AND_CHECK in more places

Summary:
Giving more details about the issue to the user

Depends on D151

Reviewers: Mathieu_Du

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

xml-formatter: De/serialize whether encoding profiles are enabled or not

Reviewers: Mathieu_Du

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

Remove obsolete Android build cruft

This is not needed any longer.

Update .gitignore

tools: Fix string leak

Only allocate the return string when we know we are going to return
it.

Coverity CID #1292292

Remove INSTALL file

autotools automatically generate this, and when using different versions
for autogen.sh there will always be changes to a file tracked by git.

ges:xml-formatter: Call g_markup_parse_context_end_parse

Summary:
Otherwise the parser context will never know that is all the XML it
will receive and fail out if the XML document is not valid (in that
case if it does not end)

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

Reviewers: Mathieu_Du

Reviewed By: Mathieu_Du

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

ges: Add debug output when get_element returns NULL

tools:launch: Print out the timeline description as an INFO not an ERROR

tools:launch: clean user facing message on wrong timeline description

Summary:
Before:

  $ ../gst-editing-services/tools/ges-launch-1.0 -p
  0:00:00.028629728  8155      0x17e1b60 ERROR                default ges-launcher.c:214:_create_timeline: serialized timeline is   -p

  ** (lt-ges-launch-1.0:8155): ERROR **: Could not create timeline, error: Could not find a suitable formatter
  [1]    8155 trace trap (core dumped)  ../gst-editing-services/tools/ges-launch-1.0 -p

  $

After:

  $ GST_DEBUG=0 ges-launch-1.0 -p

  ERROR: Could not create timeline, error: Could not find a suitable formatter

  $

Reviewers: Mathieu_Du

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

nleobject: It is wrong to update object->stop in set_property.

Summary: It must only be done when the object is commited.
We can do that in constructed though, as the changes will
anyway be commited when the object is added to a composition.

Also update the tests, as we set properties spearately then
check the stop, we can commit the source at its creation without
removing meaning from the tests.

Reviewers: thiblahute

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

timeline, track: Emit commited at the correct moment.

Summary:
+ [API] GESTrack::commited signal.
+ [API] ges_track_commit_sync

We were emitting commited when timeline_commit was called, which
wasn't very helpful. This commit makes it so we emit commited once
all the compositions have actually been commited.

We also add a synchronous commit method to spare the user
the need to connect to the signal and wait, and update the
documentation.

Reviewers: thiblahute

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

layer: call timeline_element_set_timeline in layer_set_timeline.

Summary:
Otherwise if there was still a reference to the layer when it
is removed from the timeline, it fails when the last reference
is released, because timeline_element_set_timeline calls
timeline_remove_element, which tries to remove the element from
an already disposed hashtable.

Reviewers: thiblahute

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

tests: Use AM_TESTS_ENVIRONMENT

Needed by the new automake test runner

ges-launch: Add support for +test-clip

Summary: With the pattern as a mandatory argument.

Reviewers: thiblahute

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

ges-project: Surface a meaningful error when no suitable formatter.

Reviewers: thiblahute

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

ges-launch: Add a save-only option.

Summary: + Allows to serialize the timeline without playing it back.

Reviewers: thiblahute

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

Automatic update of common submodule

From bc76a8b to c8fb372

timeline-element: Disconnect child properties handlers.

Summary:
+ And freeze notifies while doing so.

We had a race with GstController which isn't MT safe, we can
fix it by propertly disconnecting signals, and making sure
no notifies are emitted while doing so.

Reviewers: thiblahute

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

track: Set any caps features on tmpcaps.

Summary: Before checking if we have a specific constructor for a track type.

Reviewers: thiblahute

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

pipeline: no reason to disconnect a pad that is NULL anyway.

Reviewers: thiblahute

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

ges-launch: Better document options.

+ Sort them by topic
+ remove --sample-paths and --sample-paths-recurse.

http://phabricator.freedesktop.org/D58

ges-launch: port to GApplication

Summary: + Extract some utility functions.

Reviewers: thiblahute

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

build: no reason to introspect nodist sources.

Summary: g-ir-scanner was erroring like crazy on the generated sources.

Reviewers: thiblahute

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

ges-pitivi-formatter: Don't flood stdout with alarming conclusions.

Reviewers: thiblahute

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

completions: port to new base gst script

completions: Fix completions after the first command.

Revert "ges-launch: no need for a tmp string pointer"

This reverts commit 44a0924c1f6b07f0c91ee8bd03d3ae5d97da92d5.

There indeed is a need for a tmp string pointer.

ges-launch: no need for a tmp string pointer

ges-launch: free string before going out of scope

CID #1291632

build: fix make distcheck.

ges: Do not clear potentially NULL errors

And avoid dereferencing NULL errors

ges-launch: Prefix clip, transition and effect instruction with a +

Slightly changing the CLI so that we have indicators of the timeline
commands adding new objects.

ges: Accept path as URI in the create clip structured interface

ges-launch: Remove useless options, rename some short options.

bash-completion: Add support for new ges-launch commands.

parse.l: Modify command arguments.

+ --clip uri=file:// becomes clip file:// for example.

ges: command-line-formatter: Properly error out on invalid arguments

ges: Factor out a GESCommandLineFormatter class

This formatter will allow any user to deserialize a timeline using
the new ges-launch command line interface

structured-interface: Be clever when no layer priority specified.

And add the new element to the same layer as the last clip that
was added, insted of adding to the last layer of the timeline
(and with the current code, actually adding a new layer each time)

ges-structured-interface: fix build

ges: Automatically put clips at the end of layer if no start specified

In the 'structured' interface we should add it at the end of the layer

And make use of the new ges_timeline_get_layer API

structure-interface: rename layer-priority to layer.

+ And add a short name for it in ges-launch.

ges: Add layer up to the wanted layer priority in the structure interface

Making the thing easier to use

ges-launch: parse property names longer than 1 char.

+ And finish the previous structure when encountering a setter.

ges-launch: Update lexer / parser to handle set-*

+ cleanup of the lexer

tools: Implement a new CLI interface for the timeline creation

ges: Handle setting child property on container directly in the structured based interface

launcher: Add support to --set-property in the parser

ges-launch: Implement a new parser for the commandline.

Summary: + flex-based lexing and manual simplistic parsing.

Test Plan: Use that stuff to make awesome things, see if it breaks.

ges: Add an internal GstStructure based interface

To be use by GstValidate action and ges-launch

Reviewers: Mathieu_Du, thiblahute

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

ges: Fix build for older GLib

The return type of g_hash_table_insert changed from void to boolean

container: implement children property handling

ges: Move the notion of children properties to GESTimelineElement

Summary:
Deprecate the old GESTrackElement children property handling API.

New APIs:
  * ges_timeline_element_list_children_properties
  * ges_timeline_element_lookup_child
  * ges_timeline_element_get_child_property_by_pspec
  * ges_timeline_element_get_child_property_valist
  * ges_timeline_element_get_child_properties
  * ges_timeline_element_set_child_property_valist
  * ges_timeline_element_set_child_property_by_pspec
  * ges_timeline_element_set_child_properties
  * ges_timeline_element_set_child_property
  * ges_timeline_element_get_child_property
  * ges_timeline_element_add_child_property
  * ges_timeline_element_remove_child_property

Deprecated APIs:
  * ges_track_element_list_children_properties
  * ges_track_element_lookup_child
  * ges_track_element_get_child_property_by_pspec
  * ges_track_element_get_child_property_valist
  * ges_track_element_get_child_properties
  * ges_track_element_set_child_property_valist
  * ges_track_element_set_child_property_by_pspec
  * ges_track_element_set_child_properties
  * ges_track_element_set_child_property
  * ges_track_element_get_child_property
  * ges_track_element_add_child_property

Reviewers: Mathieu_Du

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

timeline: Add API to find a layer with a specific priority in a timeline

Summary:
API:
  * ges_timeline_get_layer

Test Plan: Nan

Reviewers: mathieu.duponchelle

Revert "ges: Move the notion of children properties to GESTimelineElement"

I got some trouble with

  arc land

and I wanted to push the 3 commit coming after this revert as 3
different commits but they ended up being all squash into one single
commit, which is clearly not cool for later bisecting and blaming.
Reverting that commit and re pushing those 3 commits as they were
supposed to be.

This reverts commit 9fe15ef4354dc1d878dbdec80908ac8541bc6131.

ges: Move the notion of children properties to GESTimelineElement

Summary:
Deprecate the old GESTrackElement children property handling API.

New APIs:
  * ges_timeline_element_list_children_properties
  * ges_timeline_element_lookup_child
  * ges_timeline_element_get_child_property_by_pspec
  * ges_timeline_element_get_child_property_valist
  * ges_timeline_element_get_child_properties
  * ges_timeline_element_set_child_property_valist
  * ges_timeline_element_set_child_property_by_pspec
  * ges_timeline_element_set_child_properties
  * ges_timeline_element_set_child_property
  * ges_timeline_element_get_child_property
  * ges_timeline_element_add_child_property
  * ges_timeline_element_remove_child_property

Deprecated APIs:
  * ges_track_element_list_children_properties
  * ges_track_element_lookup_child
  * ges_track_element_get_child_property_by_pspec
  * ges_track_element_get_child_property_valist
  * ges_track_element_get_child_properties
  * ges_track_element_set_child_property_valist
  * ges_track_element_set_child_property_by_pspec
  * ges_track_element_set_child_properties
  * ges_track_element_set_child_property
  * ges_track_element_get_child_property
  * ges_track_element_add_child_property

Reviewers: Mathieu_Du

Reviewed By: Mathieu_Du

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

ges: Remove all reference to already dead GESSimpleLayer

project: remove unnecessary dereference

g_clear_error() already dereferences the error pointer, no need to manually
check and do it.

CID #1257630

examples: check argument is valid

ges-base-xml-formatter: fix setting of child properties

Make sure all child properties get set. GstStructureForeachFunc
takes a gboolean return value that decides whether to
continue or not.

ges-meta-container: fix ges_meta_container_foreach()

Really call function on all metadata inside the container
instead of stopping randomly. GstStructureForeachFunc
takes a gboolean return value.

ges-base-xml-formatter: fix property setting

GstStructureForeachFunc has a gboolean return value,
and the foreach function will stop unless we return
TRUE here. This meant it was potluck whether all
properties in the structure got set or not.

Fixes setting of text overlay clip text property
in particular.

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

build: fix make distcheck.

And install bash-completions in the supplied prefix.

ges-launch: enable auto-completion.

Summary: + And be a little smart about it.

Test Plan: New feature, working, not testing bash completion

Reviewers: tsaunier

Differential Revision: http://internal.opencreed.com:8888/D25