upgrade to 1.2.4

Change-Id: Ie11e708a3b81c878dfd2ca2dc83a9ecbf43bc991

add packaging files

Change-Id: I6bf89cdea193498b52f3ed27a6f52c342f5590c9

Release 1.2.4

Update .po files

structure: error out when trying to fixate a fraction near an invalid target

baseparse: Fix memory leak

Queued frames were not released after being pushed, this
caused a leak of the GstBaseParseFrame structure.

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

gst-launch.1: Playbin2 is dead, long live playbin

Looks like that was the last remaining mention in core ...

pad: Include event type in debug output when delaying a sticky event because of not-linked

launch: place the deep-notify on the right pipeline

If the toplevel bin is not not a pipeline, we place the bin in a
pipeline. Also make sure that we connect to the deep-notify of this new
pipeline because we will g_signal_handler_disconnect() from it later.

docs: fix multiqueue docs for new template names foo_%d -> foo_%u

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

ghostpad: use gst_pad_get_peer to acquire a reference to the target pad

This ensures that the lock of the internal pad is held while referencing
it's peer (= the target pad), which ensures that the peer is not
going to be unlinked/destroyed in the meantime.

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

ghostpad: hold a reference to the target pad while unlinking it

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

buffer: remove wrong comment

Refcount and writability are not related for memory objects.

parse: Don't dist the bison and flex generated headers

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

docs: Fix typos and remove unknown annotations

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

queue2: don't truncate the temp file on shutdown

We want to keep the downloaded file untruncated so that we can use it
again later.

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

queue2: Fix merging of ranges

Make a method to get the seeking threshold. If data is further away from
this threshold we want to perform a seek upstream.
When the current downloaded range can merge with the next range,
actually include the data of the next range into the current range
instead of discarding it. Also decide if we seek to the write position
of the merged range or continue reading.

error: GST_RESOURCE_ERROR_NOT_AUTHORIZED will be available since 1.2.4

error: Add RESOURCE_NOT_AUTHORIZED error

This allows to distinguish normal read failures from read failures
where we miss authorization.

Release 1.2.3

Update .po files

multiqueue: do not reduce single queue below current level

When the single queue size was just bumped by 1 to allow more buffers to
be added, the buffers limit could be reduced to the current level when
setting the max-size-buffers property. This would result in a stall
since the queue would not grow anymore at this point.

Prevent this by not reducing a single queue size below the current
number of buffers + 1.

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

po: Update translations

pad: fix sticky event leak after sticky_events_foreach

events_foreach adds an extra ref when giving the event to the
user function. In case it was unrefed by the user, this extra ref
disappeared, but events_foreach still should unref again to
lose its own ref before removing the event from the array.

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

multiqueue: prevent buffering forever with playbin

When prerolling/buffering, multiqueue has its buffers limit set
to 0, this means it can take an infinite amount of buffers.

When prerolling/buffering finishes, its limit is set back to 5, but
only if the current level is lower than 5. It should (almost) never be
and this will cause prerolling/buffering to need to wait to reach the
hard bytes and time limits, which are much higher.

This can lead to a very long startup time. This patch fixes this
by setting the single queues to the max(current, new_value) instead
of simply ignoring the new value and letting it as infinite(0)

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

segment: gst_segment_offset_running_time() will be available in 1.2.3

tests: improve check, also check stream-time

tests: add unit test for segment _offset_running_time()

Add a unit test to check that positive and negative offsets are applied
correctly in various cases.

pad: use new segment offset method to apply the offset

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

segment: add method to offset the segment running-time

Add a method that can apply an offset to the calculated running-time of
a segment.

baseparse: do not ignore TIME segments

TIME segments are being ignored and a standard initialized
segment is used instead. This causes issues as not properly detecting
reverse playback or not cliping output based on the segment.

This seems to be a regression from one of the GstSegment/GstEvent
redesigns on the 0.10 -> 1.0 transition

segment: take offset into account in _to_position()

Take the offset into account when converting between running-time and
segment positions.

collectpads: take offset into account for expected segment position

The firt valid segment position is start + offset.
Also add some more debug and a FIXME

tests: baseparse: add test for reverse playback on passthrough

Baseparse stores buffers for reverse playback to push on the next
DISCONT, the issue was that it wouldn't ever check for a discont
on passthrough mode as it skips all real parsing. This test
was create to verify this issue and prevent it from happening again

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

tests: baseparse: add basic test for baseparse

Just a small test to check that basic playback works

baseparse: do not accumulate buffers on passthrough mode

If on passthrough during reverse playback, do not accumulate buffers as
baseparse will never check for DISCONT flag to push those buffers.
So just push buffers downstream as if it was forward playback.

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

baseparse: remove pending_segment as it was being misused

It wasn't required, instead baseparse was using it to check the media
caps to identify if it was handling audio or video.

The pending_segment was removed and a checked_media boolean
replaced it for a more accurate naming.

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

baseparse: push pending events before GAP event

A GAP event is handled as an empty buffer by sinks and they expect
to receive start up events before GAP events (like a segment).

This is important specially if there is a GAP at the beginning of
a stream (before any buffers) so that the segment event can be
pushed downstream before the GAP

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

baseparse: refactor pending events pushing

Refactor code repeated 3 times to a common function

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

docs: add missing files for distribution

 * add some documentation files in docs/design
 * add docs/list-ulink.xsl so check in docs/manual works

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

multiqueue: Fix hanging if shut down while handling a serialized query

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

pad: Add unit test for adding/removing blocking probes while a pad is blocked

And make sure that these new probes are actually called if they should
instead of silently blocking the pad forever.

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

pad: Check if new probes need to be called when adding/removing some

This allows blocking a pad, add a new blocking probe, removing
the first probe and then having the second probe called. Which
could then decide that data-flow should actually continue
instead of blocking now.

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

tee: Remove dyn lock

It was used for pad-alloc in 0.10 but currently is completely unused
and not necessary. All pad access is protected by the tee object lock
and keeping another reference to the current pad.

tee: Keep another ref to our one and only srcpad around while pushing

A pad probe on that pad might otherwise just release the pad, drop
the last reference and cause great misery.

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

pad: Keep an extra ref of the pad when calling an IDLE probe immediately

The callback might destroy the pad.

pad: Only call IDLE probes if we are actually idle

Also only check the data types for non-IDLE probes. When we
are idle, we have no data type obviously.

Previously we were calling IDLE probes during data flow whenever
a non-blocking probe would be called. The pad was usually not idle
at that time.

pad: Don't ignore probe callback return value when immediately calling IDLE probe

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

scripts: git-update.sh: fix for non-master branches

Pull from tracking branch instead of origin/master, so
that this works with e.g. 1.2 as well.

Release 1.2.2

Update .po files

po: Update translations

funnel: Setting the PROXY_CAPS flag on the srcpad does not make much sense

funnel outputs whatever one of the upstreams currently outputs, a caps
query to a random upstream does not give the right answer here.

funnel: Proxy CAPS and ALLOCATION queries

multiqueue: post 100% buffering if single queue is not linked

This makes buffering stop in case a stream switch happens. This is
important for adaptive streams that can disable not-linked streams
to avoid consuming the network bandwidth.

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

queue: don't ignore event return value

Pass the event return value upstream.
Remove strange goto construct.

tests: fix GstBus unit test with latest GLib

g_source_remove() works on the default main context, and
we're doing things with a custom context. Fixes warning
with newer GLib versions.

tools: gst-launch: don't try to remove already-removed GSource from main loop

It's considered a programming error in recent GLib versions now.
We may already have removed the source by returning FALSE from
the callback if it was fired. Fixes warning with newer GLibs
when interrupting a pipeline with Control-C.

multiqueue: Wake up on reconfigure event

After patch bda406c4, the state of the singlequeue was set to OK, but nothing
would then wake up the thread, as the other wakeup functions only look at
singlequeues that are marked as having received as not-linked.

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

gstpad: drop assertion on gst_pad_peer_query_position

It is a 'both' query, so it can be sent both ways

inputselector: handle gap events

Use gap events to advance the selector's pad position.

This is relevant to keep sync_streams mode working when one of the
streams doesn't have data all the time.

Revert "ghostpad: copy sticky events to SRC ghostpads"

This reverts commit 8162a583a4dd68582bf186e2e47a8f0d68fa1980.

Automatically copying the sticky events makes it impossible for apps
and elements to filter the events with event probes. This causes
regressions (See #719437). The best option is to let the app/element
copy and filter the events themselves after the ghostpad target is
set.

bin: Resync iterator if necessary

baseparse: ensure to preserve upstream timestamps

... rather than have subclass coming up with an internally parsed one.

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

ghostpad: copy sticky events to SRC ghostpads

Update the sticky events on SRC ghostpads when retargeting. This ensures
that the ghostpad has the exect same sticky events as the target pad. We
don't want to do this for SINK ghostpads, they got the events from
downstream and we don't want to overwrite them with the target pad
events.

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

gstutils: Escape stream id format in comments

These must be escaped for gtk-doc to parse the comments without warnings.

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

query: Fix gi annotations of gst_structure_new_custom()

gst-launch: exit with an error code when an error occured

If the pipeline failed to pre-roll or the user interrupted the
execution then set the exit code to a positive value.

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

pluginloader: check read/write before closed

first try to read or write on the socket before checking the closed state. This
makes sure we handle all data on the socket before erroring out.

poll: improve debug

So that we can see the return values of functions in the log.

Release 1.2.1

Update .po files

po: Update translations

baseparse: try first frame pts and dts for a valid start timestamp

baseparse: print proper variable in debug statement

segment: resurrect sanitizing start and stop for seeking

basesrc: mind boggling wrap when comparing offsets

typefind: use g_get_prgname() for error message

valve: proxy caps and allocation

Proxy the caps queries on the srcpad as well.
Proxy the allocation query on the sinkpad.

datetime: Make sure to include gst_private.h before glib-compat-private.h

We need to define the GLib log domain before including glib.h, which is
included by glib-compat-private.h.

parse: Fix transfer annotations for parse_launch functions.

gst_parse_launchv, gst_parse_launchv_full and gst_parse_launch_full
all return floating refs, the same as gst_parse_launch, which just
calls gst_parse_launch_full internally anyway.

Add a unit test assertion to check it's true.

Spotted by nemequ on IRC.

collectpads: Call the collected function while it returns FLOW_OK

This allows us to make sure the elements is EOS and does not have
remaining buffers to be drained.

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

core: Fix max DEBUG_LEVEL incongruence on 5 vs 9

In the docs and the autocompletion logic the maximum
value jumped incongruently between 5 and 9.

docs: Fix some reference URIs

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

pluginloader: Check errors on the proper fd

Most likely a copy-paste error from the block before.

If we're going to check for error/closed on the write fd... do it
on the write fd

Automatic update of common submodule

From 6b03ba7 to 7412249

Release 1.2.0

Update .po files

context: Add test for the context caching in GstBin

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

Revert "Potential GstContext regression"

This reverts commit e658379534eb4a90b654d90f1d0bdf86f37c6e31.

This test commit should've never been pushed. Oops.

bin: Make sure to cache context types that we did not store yet

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

Potential GstContext regression

Since the refactoring of GstContext (commits
qc9fa2771b508e9aaeecc700e66e958190476f,
a7f5dc8b8af837f01782d1572379948ff62daab7,
690326f906dc82e41ea58b81cdb2e3e88b754,
d367dc1b0d4ecb37f4d27267e03d7bf0c6c06a6, and
82d158aed3f2e8545e1e7d35085085ff58f18) I am no longer able to get
a shared context for an element that is used twice in a pipeline.

I used the documentation and eglglessink as my reference for
implementing the GstContext logic.

As the code was tied to a hardware decoder, I have ported the
GstContext code to fakesink to show the problem. Using the old
API a single ExampleMgr instance is created, but using the new
API each element is creating its own instance.

collectpads: Make sure that the object lock is always taken when accessing the private pad list

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

collectpads: Use private pad list in set_flushing_unlocked

pads->data is the public list. It is dynamically rebuilt at each call to
check_collected, in check_pads to be specific. When you add a pad and
collectpads have been started, it is not added to the public list.

Thus there exists a possible race where :

1) You would add a pad to collectpads while running.
2) You set collectpads to flushing before check_collected has been called again
-> the pad is not set to flushing
3) the pad starts pushing data as downstream might not be prepared, in the case
of adder it then returns FLOW_FLUSHING.
4) elements like demuxers, when they get a FLOW_FLUSHING, stop their tasks,
never to be seen again.

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

tests: handle unscheduled entries correctly

Make the testclock return GST_CLOCK_UNSCHEDULED when an unscheduled entry is
used for gst_clock_wait() or gst_clock_wait_async().

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

gst-uninstalled: Allow specifying the checkout directory by env variable

For some rare cases, one might not be able to use the hardcoded $HOME/gst
location yet would still want to use the gst-uninstalled script as-is (which
has the benefit of being constantly updated).

For these cases, the checkout directory can be specified with the
GST_UNINSTALLED_ROOT environment variable.

Ex:
export GST_UNINSTALLED_ROOT=$HOME/somewhere/with/checkouts

And then just call gst-uninstalled directly:
$GST_UNINSTALLED_ROOT/gstreamer/gst-uninstalled

Automatic update of common submodule

From b613661 to 6b03ba7

Automatic update of common submodule

From 74a6857 to b613661

Automatic update of common submodule

From 12af105 to 74a6857

check: testclock: fix function guards

Should be g_return_*() not g_assert(), even if it's for tests only.