validate: launcher: Allow checking if bugs linked to blacklist is fixed

validate:launcher: Consider wanted tests as whitelisted

validate: meson: Do not mixup gstvalidate lib and tracer plugin

Even though it is mostly the same thing in the end

validate:launcher: Timeout if running gdb takes too much time

validate:launcher: Allow running the testsuite N number of times

validate: Remove extra buffering status prints

validate:launcher: Fix usage in an uninstalled environment

validate: Use gst_debug_get_stack_trace instead of our implementation

And remove now useless config.h.meson file

validate:launcher: Try to generate a backtrace on segfaults

validate:launcher: Port to Python3

And sync logging.py with Pitivi version

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

Avoiding problems when using subproject:

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

validate: report: Do not unref an object where we do not own a ref

g_io_stream_get_output_stream is transfer none

validate: gir: We should not depend on GstVideo

We do not depend on it at all in GstValidate itself

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

validate: Allow using json-glib as a subproject

validate: media check: Always print runner infos when bailing out.

validate:launcher: Do not try to set DISPLAY envvar to None

meson: update version

Back to development

Release 1.10.0

scenarios: Simplify Makefile

scenarios: Add a forward key-unit trick mode scenario

Not enabled for the time being

baseclasses: Also check for minimum number of video-tracks

Some scenarios might only be for video files and are meaningless for
audio-only files

meson: Don't depend on gstreamer-check-1.0 on windows

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

Revert "meson: move gstreamer-check-1.0 dependency to validate/tests/check"

This reverts commit e8e51bdad499b38d2acc0216dc124bb82b0bd72b.

Does not actually work. See:
https://bugzilla.gnome.org/show_bug.cgi?id=773114#c31

meson: move gstreamer-check-1.0 dependency to validate/tests/check

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

validate: Blacklist failing hls tests

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

Also fix link issue in the plugin with libdw

Fix non-C89 code

In the automake build system we force C89 which does not allow mixing
code and declaration.

Fix missing linker flags for libdw

This was added in the meson build but was only checked and not used in
the automake build.

debug-viewer: window: add helper to get visible range

Move this code to the window class, as multiple plugins are going to need it.

degbug-viewer: models: only temporarilly modify the row for filtering

This avoid that we have to chek the type in the getter.
Also update the comment - we need the strip since the readline call
will not strip the newline.

gst-devtools/codecanalyser: Fix typo NO_CONFIGURE should be NOCONFIGURE

For consistency with the rest of the autogen.sh scripts.

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

debug-viewer: app: Switch for rc_parse to css

This gets us the line shading back and some size savings.

debug-viewer: models: allow filter to check COL_MESSAGE

COL_MESSAGE contains the message offset as an internal optimization. When
preparing a row for filters, we need to replace this. Otherwise filters
get an 'int' instead of the 'string' they expect.

validate: fix typo

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

validate: fix typo in gst-validate-report.c

parametter -> parameter

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

debug-viewer: small code cleanups

Inline a few statements. Remove unused variables.

debug-viewer: add a few doc strings

meson: Setup pre commit hook and fix getpluginsdir for standalone case

meson: update version

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