gitlab: revamp the bug template a bit

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tools: add a libinput analyze command with the per-slot-delta subcommand

I've been using this script ever since libinput record was available, might as
well ship it with libinput so I don't have to remember where it lives.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tools: fix a typo in a man page

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tools: record: fix dmi recording

Processing os-release in the same buffer that the dmi modalias used caused the
dmi to be recorded as 'dmi: "VERSION_ID=31"'. The cause for that was simply
that the dmi modalias was read but not printed until after the os-release
information was processed.

Fix this two-fold: rearrange that each part now reads and prints in
one go, and rename the buffers so we don't re-use them.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

libinput 1.15.3

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

touchpad: only reduce the slot count for ALPS serial touchpads

We're getting too many regressions on other devices for this feature and only
ALPS touchpads need it (it's a kernel driver bug). So let's limit this to
those devices only.

For example, synaptics serial touchpads don't keep the fake fingers and slot
states in sync when going from two to three fingers, causing an erroneous slot
downgrade. See
https://gitlab.freedesktop.org/libinput/libinput/issues/434#note_419912
That interferes with this code but fixing it is hard and anyway,
synaptics touchpads don't need the slot count drop.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

quirks: rename the alps touchpad quirk to note it's a serial TP quirk

This quirk only applies to serial ALPS touchpads, so let's name it
accordingly.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

touchpad: add a note to the synaptics touch restore feature

We have code in place to handle the quirky transition from two to three
fingers (where one slot ends and another one starts). We do not handle the
same issue when transitioning from three to two fingers.

This is a note only because it hasn't mattered so far, at least until
eb6ef9fe70635e7c91e from #408. And it doesn't matter anymore now either
because that code is now only called for ALPS devices.

https://gitlab.freedesktop.org/libinput/libinput/issues/434#note_419912

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: fix a coverity complaint

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

gitlab CI: drop the distribution "flavor" in favor of using the name

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

gitlab CI: split the task to run on a VM out from the host system

No functional changes here, it just makes the actual task more generic

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

gitlab CI: drop the distro name from the VM jobs

This changes rarely and it doesn't carry a lot of information anyway, at least
compared to the jobs that are specifically designed to build on various
distributions.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

gitlab CI: update to latest ci-templates

Removes the special distro "flavor" handling for arch and it gives us nicer
warnings for VM failures.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

gitlab CI: run the kvm test suite without libwacom

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: disable a bunch of tests relying on libwacom

Some of these may have a non-libwacom solution but let's be honest, you
shouldn't be skipping libwacom if you rely on tablets to be precise.

Fixes #436

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

gitlab CI: pass the MESON_ARGS through to the VM as well

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

gitlab CI: mark the template file as yaml file for vim

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

gitlab CI: don't run meson test on the scan-build job

Unfortunate side-effect of this: scan-build would store the logs in the build
dir, only for them to be immediately wiped by meson test. And that never
generated the scan-build warnings.

So this job was complaining about (minor) issues for a while, they just never
made it to the GUI as CI failures.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tools: constify the optarg handling of seats

This shuts up scan-build complaining about memory leaks in libinput
debug-events (needs the right combination of --device option and eventually
triggering usage()) and saves us a bunch of unnecessary allocations.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: remove double-assignment to a variable

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

gitlab CI: don't push to wayland web on scheduled jobs

We have a set of scheduled jobs to rebuild images and clean out old
containers, but since they're largely unsupervised (i.e. not in response to a
MR) we don't want to update the official documentation - just in case
something goes wrong.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tools: replay: mkdir /etc/libinput if it doesn't exist yet

Where we're replaying a device with quirks, those quirks will be placed into
/etc/libinput/local-overrides.quirks. For that to work, /etc/libinput needs to
exist so let's make it where required.

https://bugzilla.redhat.com/show_bug.cgi?id=1806322

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tools: record: fix fallback os-release file handling

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

gitlab CI: rename the custom build jobs

The distro we're running on is a side-effect, it's more important to see the
bit that describes what the job actually does.

And while we're there, shuffle the hierarchy a bit for less duplication.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

gitlab CI: use a separate variable for the template list

Just because I feel more comfortable with having this code in python than in
the templates.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

gitlab CI: don't hard-code the arch/alpine versions

We only have one each and they're not really versions anyway but now that it
is all generated through templates, let's be consistent with the rest of the
CI script.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

gitlab CI: generate the various default-builds through templating

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

gitlab CI: generate the VM tests through the templates as well

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

gitlab CI: generate the CI script from a template

All the distro-specific stuff is the same template anyway, so let's generate
this (like we already do in libevdev and the ci-templates).

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

gitlab CI: add a comment to explain one of the jobs

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

gitlab CI: whitespace fixes for better visual alignments

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

libinput 1.15.2

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tools: record: record basic os-release information as well

Because sometimes it's useful to know what distro a recording was made on, and
the kernel version doesn't always reveal that.

Fixes #428

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tools: record: rename a variable to just buf

Better name for re-use

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

gitlab CI: handling killing the qemu process better

If qemu has already shut down by the time we call kill, pgrep returns nothing
and we fail the script. Let's not do that. And let's replace kill pgrep with
just pkill in the process.

Let's get rid of the after_script part too, gitlab kills any process started
in the main script anyway.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

gitlab CI: print a warning when our VM startup fails

This happens on packet-3 and packet-4 atm, so let's print out a clear warning
that whatever the failure is, it's not directly related to libinput.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

touchpad: sync the initial tracking id state to the touchpad

Where fingers are down during startup we need to sync them to the known state
of the device so our slot count is correct. Otherwise, when the fingers are
lifted we will trigger the new assert for nactive_slots being less than 0.

Regression introduced in eb6ef9fe70635e7c91e8bb860b5a282f4cd3f91e

Fixes #429

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

touchpad: never reduce the slot count to 0

Where a user releases all touches during a SYN_DROPPED and then puts more than
one finger back down before we sync, we end up with nonzero fake touches but
a zero slot count. This is caused by a wrong event sequences provided by
libevdev in that case.

This really needs to be fixed in libevdev, see
https://gitlab.freedesktop.org/libevdev/libevdev/merge_requests/19

In the meantime, put a check in to ignore that case and never reduce the slot
count to 0. It still leaves us open for some issues where 3fg gestures may
stop working if the right sequences are triggered during SYN_DROPPED but
updating libevdev will eventually make that go away too.

Fixes #422

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

quirks: Add quirk for MadCatz RAT7 mouse

Signed-off-by: Victor Brekenfeld <github@drakulix.de>

tablet: don't force a proximity out while buttons are down

While buttons are down, don't let a forced proximity out happen. If the tablet
goes out of proximity normally that's fine but we don't force a proximity out.

Remains to be seen if this causes stuck buttons now on devices that rely on
the forced proximity out...

Fixes #403

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: drop a wait in favor of direct event processing

We know we should have an event here, so we might as well process it
immediately to speed the error case up.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: fix a duplicate button down where we want a button press + release

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: mark the tablets that require forced prox out as such

Because certain things are hard to test when you have to guess whether a
tablet has forced proximity out or not. Currently unused, see future patches.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: add a test case for checking EVDEV_ABS overrides

Problem: it's still not a 100% check because the way real udev handles the
EVDEV_ABS overrides ignores any that are set through udev properties only. So
we manually have to trigger the keyboard builtin for our test device which
can give us false positives (e.g. it wouldn't have detected #424). But still,
it'll alert us if the actual overridden values are different to what we
expect.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: drop the #define for the litest device rules

This was more useful when we installed multiple device rules but now it's only
one file anyway. Also, this drops the inadvertant double-dash
(e.g. 99-litest--Jo7Ji8.rules) which made the file name look like some
substitution was missing.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

udev: directly import the device groups

No need for the indirection of PROGRAM, then ENV for the output value.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

udev: *add* the fuzz-to-zero program to the udev RUN list, don't overwrite

For historical reasons, the keyboard builtin that sets the EVDEV_ABS values is
added as RUN. When we add our own fuzz-to-zero tool we must use +=, just using
an equals overwrites the existing RUN list.

The same is true for the IMPORT command we use to extract the fuzz to begin
with.

Fixes #424

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tools: record: record the EVDEV_ABS properties correctly

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

libinput 1.15.1

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tablet: fix the handling of axis updates after a forced proximity out

Where a pen was forced out of proximity and an eraser came into proximity
without axis updates on the prox-in, subsequent axis updates would trigger the
pen back into proximity. This resulted in two tools in proximity at once
though the new pen never went out of proximity

This would trigger crashes in various compositors/applications, see
https://github.com/xournalpp/xournalpp/issues/1141#issuecomment-578362497

The cause was a wrong condition introduced in ffd8c71e4e. We only need to
force the pen bit on if the current tool state is currently zero and no tool
update was sent with the axis event. In our case, the tool state is nonzero
already (eraser) and we can skip this bit.

Fixes #418

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: add helper functions for checking proximity events

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: add a comment to a test device

Just so I don't have to look up the git history next time

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tools: record: record the EV_ABS deltas along with the absolute value

It's useful to have this in the immediate recording. Example output now:

 - evdev:
    - [  2, 300309,   3,  47,       0] # EV_ABS / ABS_MT_SLOT               0
    - [  2, 300309,   3,  54,    3547] # EV_ABS / ABS_MT_POSITION_Y      3547 (+8)
    - [  2, 300309,   3,  58,      70] # EV_ABS / ABS_MT_PRESSURE          70 (-14)
    - [  2, 300309,   3,  47,       1] # EV_ABS / ABS_MT_SLOT               1
    - [  2, 300309,   3,  54,    3112] # EV_ABS / ABS_MT_POSITION_Y      3112 (+4)
    - [  2, 300309,   3,  58,      68] # EV_ABS / ABS_MT_PRESSURE          68 (-4)
    - [  2, 300309,   3,   1,    3547] # EV_ABS / ABS_Y                  3547 (+8)
    - [  2, 300309,   3,  24,      70] # EV_ABS / ABS_PRESSURE             70 (-14)
    - [  2, 300309,   0,   0,       0] # ------------ SYN_REPORT (0) ---------- +49ms

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tools: fix typo in debug-events man page

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

timer: make the timer offset error a bit more user-friendly

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tools: record: fix broken event time stamp recording

event->time ended up being an uninitialized field. Introduced in 5dc1a7e, the
event here isn't a struct input event but rather our internal event struct.
Fix this and reshuffle the time handling a bit so it's a bit more obvious
here.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tools: record: wrap the time offset handling into a helper function

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tools: refuse to replay recordings with timestamps from the future

This indicates a bug in libinput record, might as well complain about it.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tools: align the tool type names on proximity nicely

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

doc/user: Fix grammar.

Signed-off-by: Jacob Kauffmann <jacob@system76.com>

touchpad: correct a wrong slot count by the kernel

alps.c hardcodes 5 slots in the kernel but some devices only provide 2 slots
plus BTN_TOOL_TRIPLETAP, etc. Fix this by counting active slots and when the
fake finger count exceeds the active slots but is still less than the number
of slots, adjust the slots themselves downwards.

And because the new test device messes with our slot count assumptions for the
various tests hardcode that one device to return 2 slots.

Fixes #408

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: make the custom touch override methods filter-able

Let those functions return true if they handled the event or false where they
didn't. This makes it more flexible to override touches in special cases only
and fall back to the normal litest handling otherwise.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: wrap slot counting into a helper function

This is prep work for future devices that announce a wrong slot count. For the
tests this can be a problem if we rely on the correct slot count to decided
whether to run a test or not.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Improve git commit section

It wasn't clear about the Signed-of-by requirements, so this has been
improved with an example, and clarifies that it is a requirement, not
optional.

Signed-off-by: William Brown <william@blackhats.net.au>

Small improvements to contributing for accessibility.

This makes a small number of changes that can help improve onboarding
and diversity of contributors. Key to point out is that the code of
conduct is now one of the first items, to highlight the community
standards (rather than being at the bottom where it can be
overlooked). The use of the work "hack" often is off putting to
many people, so replaced with "work". Additionally, changed
the introduction to highlight a desire to be inclusive.

A follow up you probably could do, is have a "how to prepare the
development environment" aka "what depenedencies do I need for
my distro to build the project.

Much of this has come from experience working with outreach/diversity
experts, and my own experience on

http://www.port389.org/docs/389ds/contributing.html

Signed-off-by: William Brown <william@blackhats.net.au>

doc/user: explain the gitlab issue tags

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

doc/user: some minor improvements

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

evdev-fallback: Use input_event_init for fallback

Old-style field initialisation ignores the 64-bit time_t change in
Linux UAPI, which causes the structure to be incompletely initialised
on 32-bit systems with the 64-bit time_t kernel headers.

This patch uses the input_event_init helper from the original 64-bit
time_t enablement patch.

Signed-off-by: A. Wilcox <AWilcox@Wilcox-Tech.com>
Fixes: 5dc1a7ebd ("Adjust for 64bit time_t for 32bit architectures")
See-Also: libinput/libinput!346

gitlab CI: retry the VM jobs on script failure

Unfortunately the various VM jobs are timing sensitive and create a bunch of
false positives if the runners are under load and miss out on some of the
events (the tablet proximity handling is a particularly bad test here).
Let's retry on failure first to give the CI more opportunity to maybe succeed.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

doc/user: expand the contributing document and make it top-level

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tools: fix abort caused by the pointing stick integration quirk

See #400

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: switch from counting slot max to num_slots

Much more human-readable this way

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: simplify a slot count check

Smaller than 3 and greater or equal to 4 means - well, anything that's not 3.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: fix some wrong touchpad slot range checks

This was excluding tests from some touchpads due to slot count confusion.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

litest: Fix warnings about discarded qualifiers with check-0.13.0

check 0.13.0 introduced a new struct type TTest for test functions
instead of just a function. However, now the tcase_add_* functions use
`const Ttest *`, and since litest stores the test case in a `void *`,
we get warnings like the following:

../test/test-touchpad.c:7079:30: warning: passing argument 3 of '_litest_add' discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
  litest_add("touchpad:fuzz", touchpad_fuzz, LITEST_TOUCHPAD, LITEST_ANY);

To fix this, use `const void *`, which is compatible with both APIs.

Signed-off-by: Michael Forney <mforney@mforney.org>

libinput 1.15.0

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: indentation fix

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: add the test cases to the suite in the utils tests

oops...

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

doc/user: fix some HTML formatting

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

quirks: fix horizontal scrolling for Logitech MX Master 2S on bluetooth

It turns out that the MX Master 2S also has a different PID when connected
via bluetooth, causing horizontal scrolling to not work properly. Fix this,
by also adding it with the blueetooth PID (according to
https://github.com/libratbag/libratbag/blob/master/data/devices/logitech-MX-Master-2S.device
and in line with local testing) to the quirks file.

Signed-off-by: Björn Daase <bjoern@daase.net>

meson.build: re-add missing trailing slash to the HTTP link

Got lost in 61f81ec93e7ddfae79ce3d1e8069bfce97c6d6aa

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

gitlab CI: update to latest ci-templates

Rebuild all the images to make sure this still works fine.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

quirks: ignore mode switch button on Madcatz R.A.T.3 mouse

Avoid stuck buttons, so window managers won't behave buggy, for example:
* You click on one window, but click is emulated in another one
* You hover cursor over button/link but see no feedback
Based on quirk for Cyborg mouse.

Signed-off-by: Anatolii Lishchynskyi <iamnotacake@protonmail.com>

gitlab CI: fix skopeo copy

skopeo doesn't handle the destination credentials correctly

See ci-templates commit 0a9bdd33a98f05af6761ab118b5074952242aab0

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

gitlab CI: use a minimal alpine image for the commit checks

This runs at the same time as the other images being created so it'll fail if
the image itself doesn't exist yet. Since we only need pip here, let's use
alpine and install the two packages we need.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

libinput 1.14.901

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

gitlab CI: add an explanatory comment why we're hardcoding master

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

gitlab CI: add a local emulation mode to the gitlab CI script

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

gitlab CI: make the commit check errors more verbose and useful

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

gitlab CI: don't try adding the upstream remote if it exists already

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Expand the CODING_STYLE with an explanation of commit requirements

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

pad: add LIBINPUT_EVENT_TABLET_PAD_KEY for pad keys

The Wacom Cintiq 24HD and later tablets send specific key events for
hardware/soft buttons. KEY_PROG1..KEY_PROG3 on earlier tablets,
KEY_CONTROLPANEL, KEY_ONSCREEN_DISPLAY, and KEY_BUTTONCONFIG on later tablets.
We ignore KEY_PROG1-3 because starting with kernel 5.4 older tablets will too
use the better-named #defines.

These differ from pad buttons as the key code in itself carries semantic
information, so we should pass them on as-is instead of mapping them to
meaningless 0-indexed buttons like we do on the other buttons.

So let's add a new event, LIBINPUT_EVENT_TABLET_PAD_KEY and the associated
functions to handle that case.

Pad keys have a fixed hw-defined semantic meaning and are thus not part of
a tablet mode group.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

pad: switch the button mapping to one that can handle keys or buttons

No functional changes, we still use the same bits everywhere.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tools: debug-events: don't overrun the device array with too many arguments

Only the --device option was checked for argument count, not the rest so it's
easy to overrun the array by specifying too many devices.

Except: this was a theoretical bug only, more than 64 arguments trigger
an assertion in the argv processing in tools/shared.c anyway. Let's drop the
debug-events limit to 60 devices so we can at least have a test for this.

Found by coverity

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

gitlab CI: run the meson script for the VMs as well

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

gitlab CI: move the build instructions into a bash file

Extending/debugging scripts in the gitlab CI directly is a pain, the
turnaround cycle is terrible. Let's move this into a shellscript that we can
just call directly.

Bonus side-effect: if we wanted to extend the script: set somewhere, this is
now much easier to override.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

gitlab CI: add meson to junit script

This script was written by Emmanuele Bassi, copied from
https://gist.github.com/ebassi/e5296ec77ae9e0d3a33fd483b5613b09/

It converts meson test results into a junit file which we can then use to
display in the merge request GUI.

Note that as litest writes out junit files as well, some tests are reported
twice. Specifically: where litest fails the failure will be reported once
through litest itself and once by meson test. Oh well.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: write our test case results out as junit xml files

libcheck has the ability to write out XML files for test results, but
converting those into junit isn't ideal, for a number of reasons:
- junit xml is different to libcheck's xml, so not all data is available or
  useful. Especially with our litest wrappers around it.
- litest forking off tests means we have to wrap around everything anyway to
  avoid multiple forks writing to the same test file.

This is the minimal implementation since it's only user is likely the CI which
we control fairly tightly. So there are a few corners we can skip:
- no filename validation is performed by litest
- we write out a lot of junit xml files (one per litest fork). Rather than
  collating those we just rely on the CI to find the files.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

gitlab CI: apply some basic validity checks on the commit messages

Let's stop merge requests from users that don't set their git author name and
email address. Aside from it looking stange in the history it'll also make it
virtually impossible to ever find that user again should something important
arise in the future - especially if we switch off gitlab.

The rest is basic style, short subject lines, Signed-off-by lines and correct
formatting.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
[bentiss: use /usr/bin/env python3 as requested by the CI]
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>

gitlab CI: rename the container_prep state to just "prep"

So we can shove other stuff in there without it looking strange.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>