Release 1.9.90

meson: Fix gtkdoc using new meson features

debug-viewer: initialize all features from tuples

Also add a first doc string about the plugin initialisation.

formatting: run autopep8 over all files

We have a commit hook on the repo. Get all files to match the pep8 guidelines.

debug-viewer: cleanup imports in plugins

Don't use * imports. Don't rely on package level imports.

meson: Fix installing configured files

validate: Allow doting the pipeline on issue reporting

And let the user configure on what level of issues to do it
by setting the GST_VALIDATE_CONFIG env var.

Always dot on critical issues.

validate: Allow passing a GST_VALIDATE_CONFIG as a string

Instead of forcing user to put it in a file.
We are simply using the GstCaps synthax to parse it.

validate: Add a reference to the pipeline from each monitor

That will allow us to add more flexibility regarding the way
we report thing to the user and will allow us to properly make
reports per pipeline.

validate:launcher: Minor fix in returncode check

validate: Always print trace for critical issues

And take into account issue details level to generate backtrace.

validate: Add support for setting a report level for an issue type

Until now we could set report levels to the monitor, this adds support
for setting report level for the issue types too.

validate: transcoding: fix several error messages

No encoders found, no static src/sink pads found and keyunit and force-stop
error conditions.

validate: fix grammar on subtitle-file action description

validate: Dump the infos about NNEs if we can not properly determine the problem

It should anyway be pretty interesting pieces of information.

validate: fix build warning in generate_unwind_trace()

The unw_word_t type has different sizes for 32-bit and 64-bit, so using the
%lx format specifier on a 32-bit CPU leads to the following compile warning:

  CC       libgstvalidate_1.0_la-gst-validate-report.lo
gst-validate-report.c: In function 'generate_unwind_trace':
gst-validate-report.c:137:36: error: format '%lx' expects argument of type 'long unsigned int', but argument 4 has type 'unw_word_t {aka unsigned int}' [-Werror=format=]
     g_string_append_printf (trace, "%s (0x%lx)\n", name, offset);

Cast to long so the %lx fomart specifier can be always used.

validate: Avoid checking NULL pads when generating NNE reports

validate: fix compiler warnings

gst-validate-report.c: In function ‘generate_unwind_trace’:
gst-validate-report.c:116:1: error: old-style function definition [-Werror=old-style-definition]
 generate_unwind_trace ()
 ^~~~~~~~~~~~~~~~~~~~~
gst-validate-report.c:122:3: error: ISO C90 forbids mixed declarations and code [-Werror=declaration-after-statement]
   unw_cursor_t cursor;
   ^~~~~~~~~~~~

validate: Add backtraces in the reports

Printing them when the reporting all the details only

configure: Depend on gstreamer 1.9.2.1

validate:launcher: Do not use unset sent_eos variable

And rename class member to sent_eos_time as it is more accurate

meson:validate:test: Properly set paths to run launcher based tests

Adding a --validate-tools-path option to the launcher, allowing
to pass it from meson.

meson: Build GstValidate as a tracer

And add version to the GStValidate shared library

validate: tests: Add launcher based GstValidate tests

First checking the new not negotiated error reporting code.

validate: launcher: Always clean all tests at the end

Making sure that if an exception of anything happens we will
properly clean all the tests, or at least try to.

validate:launcher: Allow specifying expected tests errors

In the future instead of blacklisting tests we should define
what error is expected, and this way when the bug is closed,
we will notice, also, it will allow us to check GstValidate
error reporting itself.

validate: Pass information about GstValidate execution over a socket

Instead of trying to parsing stdout, generate json messages and
send them over a socket so that gst-validate-launcher can properly
have informations about gst-validate subprocess execution.

validate: Enhance not-negotiated errors reporting

Keeping negotation information around and trying to figure
out precisely why the elements could not negotied the caps
when we get a NOT_NEGOTIATED error on the bus giving the
user details about it.

validate:meson: Add tests

validate:launcher: Use a xunit reporter only when explicitely specified

validare: Enhance report message about wrong position

validate:launcher: Report error and exit when a testsuite could not load

Otherwise the user might end up seeing a lot of meaningless logs about
'removed' tests.

validate: launcher: Fix the condition to check if we need an http server

We could be checking if a string was in None

And use gs_string_assign when assigning the first string
without using printf like format.

validate: Indent report details

validate:scenario: Wait for ASYNC_DONE to set async state change DONE

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

meson: Bump version to 1.9.2

And unify the name of the disable_introspection option

Back to development

Release 1.9.2

launcher: Add --dump-on-failure switch

When the test fails, it can be useful to have the log files dumped
to stdout.

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

validate: Fix launching gst-validate-launcher in a meson based uninstalled env

meson: Add support for building GIR when used as subproject

Add allow project to us it as subproject too

validate: Add support for Meson as alternative/parallel build system

https://github.com/mesonbuild/meson

validate: Un-blacklist tests that are fixed

the bug reports to which they report have been closed and I can't make
them fail locally.

validate: Blacklist more ogg files

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

validate: Blacklist scrub_forward_seeking.op2b-mpeg2-wave_hd_mxf

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

validate: use new API when switching track with playbin3

Move all the implementations of 'switch-track' to
gst-validate-scenario.c while doing so.

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

validate: reporter: break cyclic references with reports

My patch fixing monitor leak (15e7f1bbfd84ce2cc5e6420fee2255c2be95e0f6)
introduced a ref cycle between GstValidateReporter and
GstValidateReport.

The reports uses its reporter so it needs a ref on it
to ensure it's stay alive. But reports are owned by
GstValidateReporter and/or GstValidateRunner.

Fix this by not taking a reference on the reporter but instead caching
its name.

Reviewed-by: Thibault Saunier <tsaunier@gnome.org>
Differential Revision: https://phabricator.freedesktop.org/D1029

validate: turn GstValidateReport to a mini object

It handles refcounting for us and will enable automatic leak checks when
using the 'leaks' tracer.

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

validate: inherit from GstObject instead of GObject

This allow us to use to 'leaks' detector to check if those objects are
leaked.

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

validate: call gst_deinit() after gst_validate_deinit()

This allows validate to clean up before the 'leak' tracer list leaked
objects.

Reviewed-by: Thibault Saunier <tsaunier@gnome.org>
Differential Revision: https://phabricator.freedesktop.org/D1231

validate: use MAY_BE_LEAKED flag

Reviewed-by: Thibault Saunier <tsaunier@gnome.org>
Differential Revision: https://phabricator.freedesktop.org/D1230

validate: fix pad leaks

Pads returned using the playbin get-{audio,video}-pad are reffed.

Reviewed-by: Thibault Saunier <tsaunier@gnome.org>
Differential Revision: https://phabricator.freedesktop.org/D1027

validate: reporter: break cyclic references with reports

My patch fixing monitor leak (15e7f1bbfd84ce2cc5e6420fee2255c2be95e0f6)
introduced a ref cycle between GstValidateReporter and
GstValidateReport.

The reports uses its reporter so it needs a ref on it
to ensure it's stay alive. But reports are owned by GstValidateReporter and/or
GstValidateRunner.
The best way I found to break this cycle is to introduce this purge
method. It's not great but the design is a bit tricky.

Reviewed-by: Thibault Saunier <tsaunier@gnome.org>
Differential Revision: https://phabricator.freedesktop.org/D1029

validate: reporter: prevent usage of destroyed runner

Fix crashes.

Reviewed-by: Thibault Saunier <tsaunier@gnome.org>
Differential Revision: https://phabricator.freedesktop.org/D1028

validate: runner: prevent hash table modifications while iterating

A GHashTableIter is invalided if the hash table is modified while we are
iterating. Prevent this by taking the runner lock.

Fix assertion warnings with
validate.file.transcode.to_vorbis_and_vp8_in_webm.Sintel_2010_720p_mkv_srt

Reviewed-by: Thibault Saunier <tsaunier@gnome.org>
Differential Revision: https://phabricator.freedesktop.org/D1026

validate: Add jpeg as known format

validate: Fix testsuite after additional check for buffer DISCONT flag

data: Fix make distcheck.

by distributing newly-added files.

Reviewed-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
Differential Revision: https://phabricator.freedesktop.org/D185

validate: scenario: deploy setup_sink_props_max_lateness config scenario for valgrind

Add to deploy setup_sink_props_max_lateness scenario.
When running gst-validate with valgrind option on the installed package, it fails to find that scenario.

Reviewed-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
Reviewed-by: Thibault Saunier <tsaunier@gnome.org>
Differential Revision: https://phabricator.freedesktop.org/D379

docs: Fix typo

Reviewed-by: Alex Băluț <alexandru.balut@gmail.com>
Reviewed-by: Thibault Saunier <tsaunier@gnome.org>
Differential Revision: https://phabricator.freedesktop.org/D681

validate:launcher: Add h265, opus and vp9 as known formats

Back to development

Release 1.9.1

validate: Fix usage for non-standard http server port

Translate the various stream_info URI from the standard port to the
specified port
Make the port option an integer option

media-descriptor: Fine-tune stream-id checking

Only expect fully identical stream-id from URI which are not local files
nor from our local http server.

Fixes issues with non-default http server port

validate: Use presence of STREAM_START to detect track switching

When doing a track switch, the only reliable way to detect that it
happened is whether a new STREAM_START arrives.
Relying on a DISCONT buffer is not satisfactory, since there might
not have been an element setting that flag upstream.

Checking whether the first buffer after a STREAM_START has the
DISCONT flag properly set should be done in parallel

tools: Make the stream-switching action more generic

This allows it to handle both playbin and playbin3

tools: Fix playbin detection

We might be using playbin3 instead

validate: Use decodebin3/uridecodebin3 when specified

Set the USE_PLAYBIN3 environment variable to use those elements instead
of legacy ones.

tools: New tool to view and compare xunit results

validate: fix one more export

gst-validate-scenario.c:183:7: error: '_gst_validate_action_type' redeclared without dllimport attribute: previous dllimport ignore

This is also declared in gst-validate-internal.h

validate: fix bogus exports for internal symbols

Hopefully fixes win32 build.

gst-validate-scenario.c:183:7: error: '_gst_validate_action_type' redeclared without dllimport attribute: previous dllimport ignored
gst-validate-scenario.c:286:1: error: '_action_check_and_set_printed' redeclared without dllimport attribute: previous dllimport ignored
gst-validate-scenario.c:298:1: error: 'gst_validate_action_is_subaction' redeclared without dllimport attribute: previous dllimport ignored
gst-validate-scenario.c:305:7: error: '_gst_validate_action_type_type' redeclared without dllimport attribute: previous dllimport ignored

validate: Fix Validate plugin paths mixup

validate: fix typo in debug message

validate: fix some recurring typos

validate: fix validate-transcoding option descriptions

validate: improve run-time option descriptions

validate: No execute ON_ADDITION if a previous action has a playback-time

As this is what user will expect in this case.

For example with this scenario:

    set-state, state=null; playback-time=5
    set-property, target-element-name=dvbsrc0, property-name=delsys, property-value=11
    play;

validate:launcher: Add support for relative path while providing file path

Instead of providing full absolute path while validating the file, should be
able to provide the relative path with respect to the present directory.

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

validate: fix monitor leak when doing frame analysis

The monitor returned by gst_validate_monitor_factory_create() was never
unreffed.
Report instances now have to keep a ref, as suggested by the TODO, as
the reporter is no longer leaked.

Reviewed-by: Thibault Saunier <tsaunier@gnome.org>
Differential Revision: https://phabricator.freedesktop.org/D1012

validate: redefine default cat for the runner

The gstvalidate_debug may not be initialized like with the
validate/reporting which was crashing when run with GST_DEBUG=5.

Reviewed-by: Thibault Saunier <tsaunier@gnome.org>
Differential Revision: https://phabricator.freedesktop.org/D1004

validate: transcoding: fix encoding_profile leak

Reviewed-by: Thibault Saunier <tsaunier@gnome.org>
Differential Revision: https://phabricator.freedesktop.org/D980

validate: transcoding: fix caps leaks

The 'all_raw_caps' list is never used and was just leaking caps.

Reviewed-by: Thibault Saunier <tsaunier@gnome.org>
Differential Revision: https://phabricator.freedesktop.org/D979

validate: scenario: always unref srcpad

Reviewed-by: Thibault Saunier <tsaunier@gnome.org>
Differential Revision: https://phabricator.freedesktop.org/D959

validate: media-descriptor-writer: fix pad leaks

- the pad returned by gst_element_get_static_pad() was leaked.
- unref the pad from snode when updating it, not the pad passed as
callback to pad_added_cb()

Reviewed-by: Thibault Saunier <tsaunier@gnome.org>
Differential Revision: https://phabricator.freedesktop.org/D958

validate: clean up action_types list on deinit

Reviewed-by: Thibault Saunier <tsaunier@gnome.org>
Differential Revision: https://phabricator.freedesktop.org/D957

validate: keep a ref when registering an override

_add_override_from_struct() could, in theory, register more than once
the same override so we should not transfer the ref.

Reviewed-by: Thibault Saunier <tsaunier@gnome.org>
Differential Revision: https://phabricator.freedesktop.org/D956

validate: use g_object_unref() on GstValidateOverride

Those are GObject subclasses, not GstObject.

Reviewed-by: Thibault Saunier <tsaunier@gnome.org>
Differential Revision: https://phabricator.freedesktop.org/D955

validate: add mesa-related valgrind suppressions

Reviewed-by: Thibault Saunier <tsaunier@gnome.org>
Differential Revision: https://phabricator.freedesktop.org/D954

validate: generate valgrind suppression traces

Makes fixing easier as then we can just re-use the generated trace.

Reviewed-by: Thibault Saunier <tsaunier@gnome.org>
Differential Revision: https://phabricator.freedesktop.org/D953

validate: fix typo if Xvfb is not installed

Reviewed-by: Thibault Saunier <tsaunier@gnome.org>
Differential Revision: https://phabricator.freedesktop.org/D375

pad-monitor: Don't check for DISCONT flags on buffer in pull-mode

It only makes sense in push-mode

pad-monitor: Update checks for MISSING_DISCONT

* Some SEGMENT might be updates caused by calling gst_pad_set_offset(),
  which will send the same segment but with an updated offset and/or
  based field. For those segments, we don't require a DISCONT on the
  following buffer.
* Ignore differences in flags, they aren't relevant for now to figure
  out whether the segment is an update or not
* Ignore difference in 'position', it's only meant for internal usage
  by elements.
* Changes in the end position (stop in forward playback and start in
  reverse playback) are considering updates

Furthermore, also expect a DISCONT flag on the first buffer following
a STREAM_START.

gst-validate: Avoid overreading array

If we offset the argv table as argument, we need to decrement the number
of items in the array (argc) when iterating it

pad_monitor: Add a check for buffer DISCONT flag

The first buffer after a FLUSH or SEGMENT should have the DISCONT flag
set.

launcher: set gsettings-backend to prevent weird deadlocks

It seems like some sort of forking/dconf/gtype combination can
deadlock occasionally. Setting the gsettings backend to memory
makes it go away.

Same issue: https://cgit.freedesktop.org/gstreamer/gst-plugins-good/commit/tests/check/Makefile.am?id=8e2c1d1de56bddbff22170f8b17473882e0e63f9

validate: add non-seekable fragment file seek tests to the blacklist

The file has no index and is not easily seekable, seeking in these
kind of files isn't implemented.

debug-viewer: Allow running uninstalled with symlink

When uninstalled, we look at the directory of the executable to find
the resources. This patch uses realpath in replacement to abspath so
the path get expended, and symlink are followed.

validate: Stop using deprecated method

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

scenarios: Fix scrub_backward scenarios

We will be changing states, set the handles-state variable accordingly