GitLab

This was no longer optional, leading to deadcode. This regression was
found trying to fix the unwind variant in cerbero.

https://github.com/mesonbuild/meson/pull/4169

Add new GST_API_EXPORT in config.h and use that for GST_*_API
decorators instead of GST_EXPORT.

The right export define depends on the toolchain and whether
we're using -fvisibility=hidden or not, so it's better to set it
to the right thing directly than hard-coding a compiler whitelist
in the public header.

We put the export define into config.h instead of passing it via the
command line to the compiler because it might contain spaces and brackets
and in the autotools scenario we'd have to pass that through multiple
layers of plumbing and Makefile/shell escaping and we're just not going
to be *that* lucky.

The export define is only used if we're compiling our lib, not by external
users of the lib headers, so it's not a problem to put it into config.h

Also, this means all .c files of libs need to include config.h
to get the export marker defined, so fix up a few that didn't
include config.h.

This commit depends on a common submodule commit that makes gst-glib-gen.mak
add an #include "config.h" to generated enum/marshal .c files for the
autotools build.

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

Seungha Yang authored Sep 21, 2018 and Tim-Philipp Müller committed Sep 21, 2018

Fix build on some non-US locale Windows systems

Error:
  gstreamer/gst/gstdebugutils.c(194): error C2001

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

We want this enabled by default, also in releases, but people
may want to disable this for performance-critical workloads or
on embedded devices.

Requires Meson 0.48, but the feature will be ignored on older versions
so it's safe to add it without bumping the requirement.

Documentation:
https://github.com/mesonbuild/meson/blob/master/docs/markdown/Reference-manual.md#shared_library

The cross file sets this value, and we use 'ios' in Cerbero.

And match what we do for autotools here currently.

Like in autotools.

And remove duplicate option 'poisoning' and unused 'build_tools' one.

It's checked for with #ifdef so setting it to 0 or 1
will always enable it.

Everything should be behind an option now.

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

A regression was causing the helpers to be installed in /share which
would lead to permission denied error or PolicyKit to promtp for
permission. See:

  054fa3aa meson: Use new define_variable: feature instead of run_command()

Also use host_system everywhere.

Fixes a warning.

And activate them by default as with autotools

And enable by default. Was implicitly disabled because
ENABLE_NLS was not defined.

Also yield common options to the outer project (gst-build in our case)
so that they don't have to be set manually.

Xavier Claessens authored Mar 23, 2018 and Tim-Philipp Müller committed Apr 25, 2018

Meson supports building both static and shared libraries in a single
library() call. It has the advantage of reusing the same .o objects and
thus avoid double compilation.

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

Xavier Claessens authored Apr 24, 2018 and Tim-Philipp Müller committed Apr 25, 2018

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

cc.compiles() doesn't have a 'prefix' argument (yet) and the
prefix has already been prepended to the source code snippets.

https://github.com/mesonbuild/meson/issues/2364

Modeled on the autotools build, -W flags are only
added if the compiler supports them.

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

Justin Kim authored Feb 19, 2018 and Tim-Philipp Müller committed Feb 21, 2018

For Mac OS, GST_EXTRA_MODULE_SUFFIX should be set as '.dylib'.
Otherwise, GStreamer fails to load its plugins.

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

WARNING: Trying to compare values of different types (str, int).
The result of this is undefined and will become a hard error
in a future Meson release.