meson: Set default debug level to ERROR when running from git

meson: Add Autotools changes that weren't mirrored

commits:
a7d282d27256ad1d1a55afc37d1db7f60b040089
6fdb4df0f8c8a9e39f7f7cb73ab65306fb0517f5
1aceebd67f0161806dc3b4b68488d599290f283e

check/pipeline: Make failure message more informative

This will provide maybe a bit more insight the next time it fails

check/memory: Don't leak the custom allocator

gstutils: Fix a pad leak

When requesting a pad from a template and it's already linked, this
means it was a static pad. Since we only want to return an *available*
pad, we must return NULL ... but we must also remove the reference
we got from getting that static pad.

The "No need to unref" message (which wasn't true for quite some time)
dates back from the very very very first commit introducing the 0.10
features.

check: Fix leak in queue2 test

utils: Export linear regression calculation as public function

It is useful outside the GstClock code too.

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

Remove generated gstreamer.spec file

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

docs: add dataurisrc to docs and update

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

tests: filesrc: init and clear GCond and mutex

Might otherwise leak on non-Linux systems.

tests: rewrite and enable dataurisrc test

Can't use playbin for core unit tests.

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

elements: add dataurisrc to build

Moved from -bad.

dataurisrc: fix string leak in property getter

Move dataurisrc element from -bad

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

message: Ensure that the "debug" field of error/warning/info messages is valid UTF-8

The caller might pass arbitrary data here that caused the error, and
trying to set invalid UTF-8 in a GstStructure causes it to be not set at
all. Later when trying to parse it, the field will not exist and the
return value will point to invalid memory. Prevent this by storing NULL
instead.

Also print a g_warning(), the caller should never ever do this to begin
with.

common: use https protocol for common submodule

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

scripts: create-uninstalled-setup: use https protocol to clone repos

The git:// protocol is problematic from a security perspective, as
it provides no authenticity of data. https:// also works better in
environments with restricted network connectivity.

Also add CLONE_OPTS to do shallow checkouts more easily.

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

meson: Support execinfo.h on FreeBSD by using -lexecinfo

FreeBSD supports execinfo.h and backtrace* functions, but
using them requires linking with -lexecinfo.

Requires sufficiently-new meson with #1053 fixed (post-0.36).

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

tools: Remove files to be cleaned

manpages are no longer auto-generated
cov-related files should not be there (if needed we could use gitignore)

basesink: Document the interaction between unlock() and wait_preroll()

This was totally non-obvious, the kind of big problem is that subclasses must
be able to unblock their streaming thread and continue exactly where they left off
on unpause!

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

fdsink: Block in preroll_wait on unlock

The correct behaviour of anything stuck in the ->render() function
between ->unlock() and ->unlock_stop() is to call
gst_base_sink_wait_preroll() and only return an error if this returns an
error, otherwise, it must continue where it left off!

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

element: Don't increment NULL pointers

Trivial workaround for coverity false warning.

CID 1394488, 1394487.

tools: fix distcheck and .gitignore

meson: tools: install the man pages

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

tools: ship the final man pages directly, no more man pages templates

Don't use templates for the man pages, the API version change is a rare
event, so it's not really worth keeping in place the "sed" boilerplate
to have it set at build time.

Shipping the final man pages directly also makes it easer to install the
man pages with meson (in a future commit).

Note that now all the occurrences of the programs names have the API
version as a suffix.

Traditionally the example command lines looked like:

  gst-launch ...

Now they look like:

  gst-launch-1.0 ...

This reflects the actual programs names and makes it easier to copy and
paste the example commands.

Also, the .gitignore file is adjusted not to ignore the final man pages
anymore.

You may need to clean your src/build directory before pulling in this
patch.

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

controllers: add new proxy control binding

Allows proxying the control interface from one property on one GstObject
to another property (of the same type) in another GstObject.

E.g. in a parent-child relationship, one may need to
gst_object_sync_values() on the child and have a binding (set elsewhere)
on the parent update the value.

Note: that this doesn't solve GObject property forwarding and must be
taken care of by the implementation manually or using GBinding.

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

memory: log with GST_INFO instead GST_ERROR when subclass map failed.

Add unit test to ensure that.

It can be a normal execution path to do some map trials and there is
no need to worry the user in that case.

The application has to check the return value of gst_memory_map.

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

basetransform: Ensure to set the RECONFIGURE flag again if reconfiguration failed

It might've failed just because of flushing or other things, and we
should retry again on the next possibility if something ever calls in
here again.

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

meson: add_global_arguments -> add_project_arguments

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

multiqueue: Make sure not-linked streams get woken up

When running in sync-by-running-time mode, pad groups
that have exactly 1 pad and it's not-linked might never
wake up after computing a high time, as the per-pad-group
high time was only recomputed when a pad in the group
advances.

Wake those up using the global multiqueue high-time across
all other groups instead.

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

docs: misc fixes

utils: use temp var in fallback GST_WRITE_*() macros

To make sure the value is only expanded/used once, in case
there are side effects to it, and to avoid calculating it
or looking it up multiple times if there is a calculation
or lookup involved.

baseparse: Fix previous commit

Check the correct segment format value.

parse->segment.format is the format we're outputting in,
not the upstream format. Use parse->priv->upstream_format instead,
and make sure it's set in pull mode.

baseparse: Restrict query/convert responses when demuxing

If the parser is not parsing a raw elementary stream, restrict
the position, duration and conversion query replies to
things we can sensibly answer about - especially don't do
random conversions to/from bytes.

queues: Don't return negative position queries.

When subtracting queued data sizes from upstream queries
in queue, queue2, downloadbuffer and typefind, clamp the
result to not go negative, in case upstream returned
a nonsense value that's too small (as could happen if
upstream is estimating, or just broken)

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=774293

typefindhelper: Update prototype of helper_find_suggest()

forward declaration prototype is updated to match the change in:

 5a72c23 Change some types to match their prototypes

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

win32: remove copies of generated headers

docs: remove more docbook build cruft that's no longer needed

docs/README: remove more outdated pieces of info/advice

scripts/gst-plot-traces.sh: make log parsing a bit more solid

Use grep -o to grab the log message only. This makes it work with colored log
files too. Prefilter the log to not catch tracer classes.

Update the commandline for the script in the docs.

meson: require meson 0.36 and use new `pic` arg on static libs

Removes a meson warning and some special casing we had.

Change some types to match their prototypes

Particularly note that the underlying integer type of the enum
GstTypeFindProbability is implementation dependent and may not match
guint.

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

po: update for new translatable strings

docs: remove FAQ which was moved into gst-docs module

gst-uninstalled: add GIO_EXTRA_MODULES

In case glib is installed into local prefix dir.

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

utils: faster GST_WRITE_* macros if unaligned access is possible

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

parse: better error message when linking two elements with capsfilter fails

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

Add gst_print(), gst_println(), gst_printerr(), gst_printerrln()

Useful for debugging.

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

taglist: remove `return void` in gst_tag_register

MSVC warns on this and the documentation about the warning says:

> The compiler assumes the function returns a value of type int

which is a little scary, so lets just remove the unnecessary 'return'

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

tests: add unit test for gst_adapter_prev_pts_at_offset()

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

adapter: fix distance when getting prev pts/dts at offset

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

meson: don't add_global_arguments when being built as a subproject

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

basetransform: fix pool leak when early returning in decide_allocation

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

info: add GstStream and GstStreamCollection support to gst_debug_print_object()

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

basesrc: Support PROTECTION event from application

Application may want to send PROTECTION event to the src element.

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

meson: tools: generate the targets dynamically

The three targets are the same except for input and output
files, use a loop and generate them dynamically.

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

tools: don't mention gst-feedback in man pages

gst-feedback no longer exists.

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

tools: put the examples descriptions before the commands in man page

Put the description of the example command lines before the command
instead of after them. The new way is more intuitive.

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

tools: don't start lines with single quotes in man page

When a line starts with a single quote it's treated in a special way by
man, which may result in paragraphs of the man page not rendered by the
man pager, so just avoid that.

A possible solution could have been to escape the singe quote with
a \(cq sequence but this is rather unreadable, instead the text has been
reformatted to have the problematic quoted 'ppc' string on the previous
line.

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

tools: escape dashes in the man pages

The portable way to have the dashes to be rendered as ASCII minuses is
to use the sequence backslash-dash, use this style at least for text
that can be copied and pasted (e.g. command names, file names, element
options).

Also use backslash-dash in the NAME section as suggested by lexgrog(1).

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

meson: Fir dependencies of gstnet-1.0

It depends on gst_base_dep which will pull in gst_dep

meson: Add GstBase-1.0.gir to gst_base_dep

Without this, GIR generators can't find and use it

baseparse: add since marker for new API to docs and fix win32 .def file

baseparse: expose gst_base_parse_drain

meson: Advertise dependency on gst_dep generating girs

And do not simply link to libgst as the gir information
location only exist in declare_dependecy

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

tests: Fix compile warning on mingw64

In file included from ../../libs/gst/net/gstntppacket.c:35:0,
                 from netclock-replay.c:25:
../../config.h:546:0: error: "__MSVCRT_VERSION__" redefined [-Werror]
 #define __MSVCRT_VERSION__ 0x0601

In file included from /usr/x86_64-w64-mingw32/sys-root/mingw/include/crtdefs.h:10:0,
                 from /usr/x86_64-w64-mingw32/sys-root/mingw/include/stdio.h:9,
                 from netclock-replay.c:21:
/usr/x86_64-w64-mingw32/sys-root/mingw/include/_mingw.h:220:0: note:
this is the location of the previous definition
 # define __MSVCRT_VERSION__ 0x0700

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

funnel: Make sure to only lock the stream lock once

We also only unlock it once, and otherwise have it locked forever from
this thread, causing deadlocks on shutdown later.

pool: Fix compiler warning on mingw64

gstpoll.c: In function 'release_event':
gstpoll.c:239:3: error: suggest parentheses around assignment used as
          truth value [-Werror=parentheses]

   if (status = WaitForSingleObject (set->wakeup_event, INFINITE)) {
   ^~

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

win32: update .def for new API

meson: Unset the plugin paths to generate the .gir files

Avoiding problems when using subproject:

    Failed to load plugin 'something.so: file too short

debug: Remove the Gst only based stack trace printing implementation

We now have 2 other implementations that should work better.

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

tracers: leaks: Use the new gst_debug_get_stack_trace

And remove the local implementation of it.

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

gst: Use libunwind/libdw to generate backtraces if avalaible

Making the gst_debug_print_trace function more generally useful.

API:
  + gst_debug_get_trace

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

tools: gst-launch: fix minor memory leak when failing to parse options

Commit 215cfcf99338 (gstreamer: Fix memory leaks when context parse
fails) fixes some memory leak, but in one of the newly added calls to
g_clear_error() the wrong variable was passed.

When failing to parse command line options, free the "err" variable, not
the "error" one.

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

basesink: Make sure we never drop the preroll buffer

This is cosmetic as 'late' should never be set during preroll (in pause).
Though code may evolve in the future, so this is good for preventing
potential bugs.

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

basesink: Don't nest prepare/render calls

When the first buffer arrives, we endup calling:

  ->prepare()
    ->prepare()
    ->preroll()
  ->render()

This will likely confuse any element using this method. With this patch,
we ensure the preroll take place before the first render prepare() is
called. This will result in:

  ->prepare()
  ->preroll()
  ->prepare()
  ->render()

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

basesink: fix typo in documentation

Small typo in the documentatin of gst_base_sink_set_drop_out_of_segment().
Fixing it.

basesink: Fix gst_base_sink_set_drop_out_of_segment() documentation

Also silences a GI warning.

clock_gettime.c: Use __APPLE__ instead of __MACH__

Hurd also defines __MACH__, but it does not have mach_absolute_time. Use
the more strict __APPLE__ instead.

Has also been sent upstream: https://github.com/libcheck/check/pull/65

build: Fix AM_CONDITIONAL check for clock_gettime

It was always evaluating to false, so clock_gettime.c was always being
included into libcheck. This breaks building on Hurd and causes us to
always override clock_gettime() even when it is available.

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

gst: Also include the gstdynamictypefactory.c source file in the build

gst: Install gstdynamictypefactory.h header file

Revert "baseparse: fix draining with less data than min frame size available"

This reverts commit 2e278aeb7128e8732f5324ab8c8b22a47950c80a.

Some parsers, specifically audio parsers, assume to get all remaining
data on EOS and just pass them onwards. While the idea here is correct,
we will probably need a property for this on baseparse for parsers to
opt-in.

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

parse-launch: Support linking all pads with new operator

Introduce a new operator ':' - e.g. element1 ':' element2

For example, 'uridecodebin : encodebin' -
if the encodebin has multiple profiles compatible with the
decodebin, multiple links will be created.

With '!' , after one delayed link is successfully done, the
pad-added callback is disconnected.

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

Add new basesink API to exports and Since markers

Add Since markers to the new basesink API to drop
out-of-segment buffers, and add them to the
win32 exports

dynamic types: Implement dynamic types in the registry

Implement GstDynamicTypeFactory as a new registry feature.

GstDynamicTypeFactory provides a way of registering a GType
into the registry, such that it will be registered as a dynamic
type when the registry is loaded, and then automatically loaded
if the type is needed during caps parsing.

This allows using non-core types in pad templates, by loading a
registry feature to create the GType on the fly.

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

fakesink: Add property to not drop out-of-segment buffers

Implement handling in basesink to not unconditionally discard
out-of-segment buffers and expose it as a new property on fakesink
(not unconditionally in all basesink based sinks).

The property defaults to FALSE.

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

value: Update GstValue table size for GValueArray

funnel: Always push all sticky events whenever we forward a serialized event

Otherwise downstream will have an inconsistent set of sticky events at this
point, e.g. when a TAG event is pushed and downstream wants to relate it to
the stream by looking at the current STREAM_START event.

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

tee: Set GST_PAD_FLAG_PROXY_CAPS before forwarding sticky events

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

baseparse: fix draining with less data than min frame size available

baseparse would pass whatever is left in the adapter to the
subclass when draining, even if it's less than the minimum
frame size required. This is bogus, baseparse should just
discard that data then. The original intention of that code
seems to have been that if we have more data available than
the minimum required we should pass all of the data available
and not just the minimum required, which does make sense, so
we'll continue to do that in the case that more data is available.

Fixes assertions in rawvideoparse on EOS after not-negotiated with
fakesrc sizetype=random ! queue ! rawvideoparse format=rgb ! appsink caps=video/x-raw,format=I420

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

info: Replace %p and %r in GST_DEBUG_FILE

It's useful to be able to set a name pattern for GST_DEBUG_FILE so that
the same environment variable can be used for multiple processes and
still write to different files. Especially useful if these processes
run simultaneously.

%p: Replaced with PID
%r: Replaced with random number

%p is obviously useful. %r is useful when for instance running two
processes with same PID but in different containers.

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

registry: set env GST_REGISTRY_DISABLE=yes to disable registry

If GST_REGISTRY_DISABLE=yes the registry is disabled similar to
compile time switch GST_DISABLE_REGISTRY.

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

gstvalue: Make GValueArray serializable

For instance very useful for logging GValueArray with GST_PTR_FORMAT

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

element: Allow multiple conversion specifiers for request pads

This allows pad template names like "src_%u_%u", but it does not allow
multiple specifiers of string type %s as that would lead to ambiguities.

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

pad: add no-reconfigure link check

Enable it to prevent sending reconfigure when linking elements.

Useful for autoplugging when we know caps or bufferpools shouldn't change
to save doing caps renegotiation to end up with the same final scenario.

The no-reconfigure is not a proper check, it is a flag. It is implemented
as a GstPadLinkCheck to avoid creating another gst_pad_link variant.

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

meson: update version

docs: remove app dev manual and plugin writer's guide

They have moved to gst-docs and will be maintained there in future.

Back to development