Merge branch 'more-keyboard-debugging' into 'main'

ply-keyboard: Add a little mode debug logging

See merge request plymouth/plymouth!293

ply-keyboard: Add a little mode debug logging

It seems like the terminal fd is getting read from multiple
consumers, so add a little more logging to see if it's happening
in the keyboard code.

Merge branch 'debug-key-events' into 'main'

ply-keyboard: Add new plymouth.debug-key-events option

See merge request plymouth/plymouth!292

ply-keyboard: Add new plymouth.debug-key-events option

We're currently facing a bug where keys have to be hit several
times to register for a user.

In order to make traction on that problem, this commit adds a
new plymouth.debug-key-events kernel command line option
to report in more detail the keyboard events coming in from the
terminal.

Merge branch 'rework-renderer-terminal-input-handling' into 'main'

renderers: Rework input handling a little

See merge request plymouth/plymouth!291

renderers: Rework input handling a little

We don't need to flush the terminal input buffer on key events from
the renderers, because the terminal does that itself.

Also, we should disable the terminal input when a /dev/input
device is added instead of relying on it happening the next flush
period.

Finally, we should make sure the terminal gets reenabled on close,
just so we clean up after ourselves.

Merge branch 'add-terminal-logging' into 'main'

ply-terminal: Log when terminal input gets enabled/disabled

See merge request plymouth/plymouth!290

ply-terminal: Log when terminal input gets enabled/disabled

We're currently seeing a bug where it seems like the terminal might
be getting intermittently disabled.

This commit adds some logging to confirm or rule out that possibility.

Merge branch 'initialize-line-dimensions' into 'main'

label-freetype: Initialize line dimensions to 0

See merge request plymouth/plymouth!289

label-freetype: Initialize line dimensions to 0

Now that we can print the line dimensions, we should
initialize them to zero.

Merge branch 'debug-escape-sequences' into 'main'

ply-terminal-emulator: Add way to see escape sequences

See merge request plymouth/plymouth!288

ply-terminal-emulator: Add way to see escape sequences

This commit adds a new kernel command line option
plymouth.debug-escape-sequences that makes escape sequences stay
unparsed and instead get printed on the screen.

Merge branch 'show-label-dimensions' into 'main'

label: Put dimensions of text in log

See merge request plymouth/plymouth!287

label: Put dimensions of text in log

This just helps with debugging.

Merge branch 'measure-hidden-freetype-labels' into 'main'

label-freetype: Measure control even if hidden

See merge request plymouth/plymouth!286

label-freetype: Measure control even if hidden

Sometimes we want to query the size of hidden labels too, so
it's important we still compute the size, even if hidden.

Revert "ply-utils: Use lstat instead of stat for ply_file_exists"

This reverts commit e989867f4871bba564cd5dcc0de64da8ead36058.

I had stat/lstat momentarily crossed in my brain

Merge branch 'freetype-metrics-fix' into 'main'

label-freetype: Ensure font metrics are up to date when querying dimensions

See merge request plymouth/plymouth!285

label-freetype: Ensure font metrics are up to date when querying dimensions

If code calls ply_label_get_width without ply_label_show or some other
call that forces the dimensions to be computed first, then the returned
width will be wrong.

This commit makes the freetype plugin look more like the pango plugin
where the size will computed on demand when querying the width, if
necessary.

Merge branch 'freetype-font-debugging' into 'main'

Improve font loading error debug messages in freetype plugin

See merge request plymouth/plymouth!284

label-freetype: Log font loading error

Right now if there's an error loading a font we quietly just
proceed.

This commit adds a debug log message saying what the error is.

label-freetype: Don't bother loading fallback font if it doesn't exist

If the fallback font doesn't exist, we shouldn't even try to load it,
there's no point.

Instead, put a nice error in the log.

ply-utils: Use lstat instead of stat for ply_file_exists

If a file is a symlink, we usually want to follow it, so
testing the symlink itself, is less than optimal.

This commit switches to lstat instead of stat.

Merge branch 'pango-console-viewer-fixes' into 'main'

console viewer fixes with the pango label plugin

See merge request plymouth/plymouth!283

ply-console-viewer: Make free function NULL safe

label-pango: Pass explicit width when computing height

pango will give a height that's several orders of magnitude too
big sometimes when the width is -1.

This commit checks for -1, explicitly measures the width first, and
remeasures with that width passed in to compute the height.

ply-console-viewer: Disallow zero lines

A logic error meant the console viewer can end up being zero lines.

This commit fixes that.

label-pango: Fix inverted conditional

Right now we skip sizing the control when force == true because
of an inverted conditional.

This commit fixes that.

ply-console-viewer: Fix typo

The last commit had a typo where it would never prefer console viewer.

This commit fixes that.

Merge branch 'console-viewer-crasher-fix' into 'main'

ply-console-viewer: Handle fonts failing to load better

See merge request plymouth/plymouth!282

two-step: Be more tolerant when there's no console viewer

There are still a number of places in the code where a console
viewer is expected, when it's totally possible that there won't
be one.

This commit runs through them, and does a few other style
cleanups on the way.

space-flares: Be more tolerant when there's no console viewer

There are still a number of places in the code where a console
viewer is expected, when it's totally possible that there won't
be one.

This commit runs through them, and does a few other style
cleanups on the way.

fade-throbber: Be more tolerant when there's no console viewer

There are still a number of places in the code where a console
viewer is expected, when it's totally possible that there won't
be one.

This commit runs through them, and does a few other style
cleanups on the way.

ply-console-viewer: Handle fonts failing to load better

Right now we divide by zero if the font metrics can't be determined
and tank.

This commit attempts to detect the situation and fall back to using
kernel fb console in that case.

Merge branch 'catch-sigfpe' into 'main'

main: Catch floating point exceptions in crasher handler too

See merge request plymouth/plymouth!281

main: Catch floating point exceptions in crasher handler too

Right now we catch the two big types of crashes:

1. Segmentation fault
2. Assertion failure

But we don't catch another common type, "Divide by zero"

This commit fixes that.

Merge branch 'fix-xkb-crash' into 'main'

ply-device-manager: Handle XKB failing to load more gracefully

See merge request plymouth/plymouth!280

ply-device-manager: Handle XKB failing to load more gracefully

Right now we always assume `xkb_context_new` will succeed.

This isn't necessarily true. If it fails, we shouldn't crash.

This commit fixes that.

Merge branch 'print-backtraces' into 'main'

main: Print backtrace on crash

See merge request plymouth/plymouth!279

main: Print backtrace on crash

Since plymouth can crash before the root filesystem is mounted,
and since plymouth can prevent boot from proceeding, coredumpctl
isn't always a viable option for getting backtraces.

This commit changes the crash handler to output backtrace
information to assist when coredumpctl can't.

Merge branch 'fix-freetype-crash' into 'main'

label-freetype: Handle font failing to load more gracefully

See merge request plymouth/plymouth!276

label-freetype: Fix loading debug message

The freetype plugin was calling printf instead of ply-trace when
a font failed to load. Also, it didnt list the failure reason.

This commit fixes things up a bit.

label-freetype: Handle font failing to load more gracefully

We currently crash if the font fails to load.

This commit attempts to handle the situation more gracefully.

Merge branch 'fix-console-viewer-crash' into 'main'

fade-throbber,space-flares,two-step: Don't crash if no console viewer

See merge request plymouth/plymouth!278

fade-throbber,space-flares,two-step: Don't crash if no console viewer

The splash plugins don't create a console viewer if
`plymouth.prefer-fbcon` is on the kernel command line, but expect it
to be not NULL regardless.

This commit corrects that misconception.

Merge branch 'fall-back-to-text-faster' into 'main'

ply-device-manager: Fall back to text plugin if no renderers installed

See merge request plymouth/plymouth!275

ply-device-manager: Fall back to text plugin if no renderers installed

If there's no renderers installed there's no point in continuing to
listen for DRM events and trying to load the drm plugin.

This commit just forces text mode right away in that case.

Merge branch 'font-fix' into 'main'

Fix checks for existence of vars set by fc-match

See merge request plymouth/plymouth!274

Fix checks for existence of vars set by fc-match

The first line here checked for the wrong variable and is a dupe
from two lines earlier anyway, and the second line isn't guarded
at all, which seems to cause the weird failure we've been seeing
in openQA testing.

Also add a couple more existence checks for the same variables
later.

Signed-off-by: Adam Williamson <awilliam@redhat.com>

plymouth-populate-initrd: More dependency softification

This commit makes lack of fc-match less noisy and makes
compose.dir from libX11 be optional as well.

Merge branch 'make-deps-less-hard' into 'main'

plymouth-populate-initrd: Handle xkb and fontconfig not being installed

See merge request plymouth/plymouth!273

plymouth-populate-initrd: Handle xkb and fontconfig not being installed

Some minimal installs won't have fontconfig or xkb. In those cases,
the script should continue to work even if the initramfs will be
less featureful.

This commit fixes that.

Merge branch 'fix-usage-message' into 'main'

plymouth-populate-initrd: Fix usage message

See merge request plymouth/plymouth!272

plymouth-populate-initrd: Fix usage message

It currently says the scripts name is `plymouth` not
`plymouth-populate-initrd`.

This commit changes it to use $0

Merge branch 'meson-systemd-integration' into 'main'

meson: Fix PLY_ENABLE_SYSTEMD_INTEGRATION define

See merge request plymouth/plymouth!271

meson: Fix PLY_ENABLE_SYSTEMD_INTEGRATION define

Merge branch 'fix-label-crasher' into 'main'

ply-label: Make NULL safe (again)

See merge request plymouth/plymouth!270

ply-label: Make NULL safe (again)

commit a045f01568f53ca10ef8b76d9f507ccd3546e9d8 tried to make ply-label
NULL safe, but missed a strdup.

This commit fixes that.

Merge branch 'input-device-crasher' into 'main'

ply-input-device: Fix crasher

See merge request plymouth/plymouth!269

ply-input-device: Fix crasher

If we have an error during early input device initialization we
can blow assertion because we may try to cleave it from the event
loop it is not yet attached to.

This commit fixes that.

generate-version: Use day number instead of week number

Right now the version is generated from week number but that
means more than one release in a week won't work.

This commit changes it to day number.

Merge branch 'main' into 'main'

main: fix out of bounds write

See merge request plymouth/plymouth!265

main: Fix out of bounds write

There's currently an out of bounds write when copying dmesg to to
the boot buffer.

This is because there's a newline and a terminating NUL and only
one of the two was being accounted for.

This commit fixes the problem by dropping strcat/strcpy and using
printf style functions instead.

Spotted by Ilya K.

Merge branch 'fix-generate-version' into 'main'

generate-version: Fix script inside of a tarball

Closes #232

See merge request plymouth/plymouth!268

generate-version: Fix script inside of a tarball

The heuristic to check whether or not we're inside of a git checkout
or a tarball isn't working properly.

This commit changes it to use git rev-parse --is-inside-git-dir which
is pretty specifically designed for this use case.

Closes: https://gitlab.freedesktop.org/plymouth/plymouth/-/issues/232

Merge branch 'fix-keymap-memory-leak' into 'main'

ply-keymap-icon: Fix memory leak

See merge request plymouth/plymouth!267

ply-keymap-icon: Fix memory leak

It seems like there's a memory leak with the keymap icon text if
there isn't a hit.

This commit fixes that, and renames some of the variables to make
it a little clearer what's going on

Merge branch 'handle-null-better' into 'main'

ply-label: Make NULL safe

See merge request plymouth/plymouth!266

ply-label: Make NULL safe

commit 42d07913a0b72b8607190ab31931d5460765f589 made the pango plugin
NULL safe but the same problem exists in the freetype plugin.

This commit makes things NULL safe a layer higher.

Merge branch 'revamp-freetype-plugin' into 'main'

label-freetype: Revamp to work better

See merge request plymouth/plymouth!264

label-freetype: Revamp to work better

Recent changes to the freetype plugin made it quite buggy. There
was an infinite loop bug, and also a failure in metrics handling.

There's also a number of outstanding issues:

1. Kerning is not properly applied
2. hidpi displays don't render correctly

Rather than try to tackle all of these problems in a peicemeal way,
this commit substantially reworks the code.

It introduces a ply_freetype_unit_t type for more clearly managing
the fixed point math freetype requires. This type relies on less
than well defined compiler behavior, however, so it may cause issues
down the road.

It consolidates measuring and drawing code to one function, since in
both cases, the same computation needs to happen, and they had
slightly divergent implementations before.

It consolidates font loading code, so we don't end up with two
different font face loading routines, with different dpi values.

It also changes the names of some variables to be clearer in my
mind (like changing the loaded glyph to the name "glyph" instead
of the name "slot")

ply-console-viewer: Don't crash if passed a NULL format

The splash plugins currently call

ply_console_viewer_print (... NULL);

if no prompt message is specified... This leads to crash.

Rather than fixing all the splash plugins, this commit just makes
a NULL format be a noop.

Merge branch 'synchronize-display-updates' into 'main'

ply-boot-splash: Flush display updates at fixed framerate

See merge request plymouth/plymouth!263

ply-boot-splash: Flush display updates at fixed framerate

At the moment the splash plugins have a lot of leeway when
updates can go to the screen. They can pause and unpause
updates by themselves, and draw at any point.

This rope has a little too much slack, it's actually kind
of complicated for the splash plugins to manage drawing
when they have more than one moving part.

For instance, a spinner animation may be drawing autonomously
from the splash plugin itself. To avoid flicker, everything
needs to be synchronized.

This commit adds that synchronization a layer higher than
the plugins themselves, in ply-boot-splash. It accumulates
drawing updates continuously without doing any drawing, until
a given deadline, then flushes the updates out all at once.

Revert "ply-pixel-display: Pause the display and unpause it at a framerate"

This reverts commit 815d3c72e2e4f7299490e43147972e7d31b91d17.

This idea of pausing and unpausing updates at a given framerate is good,
but doing it at the ply-pixel-display level causes a proble with
monitor hotplug.

Merge branch 'fixconsoleviewerrefresh' into 'main'

splash plugins: Better handling the redraw of the console viewer when visible,...

See merge request plymouth/plymouth!257

splash plugins: Better handling the redraw of the console viewer when visible, and the splash state changes

ply-pixel-display: Pause the display and unpause it at a framerate

ply-entry: Don't crash hiding an entry that is already hidden

ply-console-viewer: introduce ply_console_viewer_clear_line ()

ply-console-viewer: Reorder the labels.

When there is less than 1 screenful of lines, the first line now appears at the top, more similar to a terminal emulator

ply-terminal-emulator: Don't reject empty lines

Merge branch 'nullify-borrowed-bytes' into 'main'

ply-buffer: Nullify bytes outside scope block

See merge request plymouth/plymouth!262

ply-buffer: Nullify bytes outside scope block

ply_buffer_borrow_bytes is a new macro to make it possible to
temporarily alter the allocation of the underlying bytes of
a `ply_buffer_t` outside of `ply_buffer_t` specific methods.

This macro works by forcing the caller to use a scope block
to clearly delineate where the allocation modification is
occurring. This macro was inspired by the python `with` feature.

Unfortunately, the macro leaves the bytes initialized outside
the scope block, so there is some temptation to continue messing
with the allocation when it's not allowed.

This commit improves the macro to address that problem by nullifying
the passed in variables at the conclusion of the borrowing scope
block.

Merge branch 'resilientunicode' into 'main'

ply-terminal-emulator: Handle incomplete Unicode characters with more resiliency

See merge request plymouth/plymouth!261

ply-terminal-emulator: Handle incomplete UTF-8 characters better

This commit introduces a state machine to better handle when part
of a UTF-8 character comes in immediately, and the rest of it
comes in later.

It also tries to better handle cases where control characters
are interleaved in the middle of UTF-8 characters.

ply-terminal-emulator: Rename PLY_TERMINAL_SPACES_PER_TAB to PLY_TERMINAL_EMULATOR_SPACES_PER_TAB

ply-rich-text: Remove now unneeded definiton

utils: Rework UTF-8 handling

ply_utf8_character_get_size currently has this odd argument at
the end that is often just set to PLY_UTF8_MAX_CHARACTER_SIZE
and also the function returns magic values for cases where it
can't figure out the size because the byte isn't a leading
byte or is otherwise not valid UTF-8.

That means that API has a nuance to it that makes the code hard
to follow at a light read.

This commit attempts to improve the situation by dropping the
extra argument, and adds a way to get the type separate from the
size for clarity.

At the same time, this commit updates all the callers to use the
new API. There are two cases where the callers are trying to
remove the last character from a UTF-8 string, so this commit
adds a new function to consolidate that logic as well.

ply-buffer: Make ply_buffer_clear fast if called multiple times in a row

ply_buffer_clear memsets over the entire allocation every time it is
run.

This commit makes it only clear the used part of the allocation.

buffer: Add APIs for editing bytes in buffer

This commit adds a few functions for getting at and
using the size, bytes, and capacity of a buffer.

This will make it easier to modify utf8 string on the side and
and update the buffer size afterward.

Merge branch 'fixvtleds' into 'main'

Set the VT keyboard state with KDSKBMODE so that the VT console doesn't change the LED state

See merge request plymouth/plymouth!259

ply-input-device: Handle the keyboard sequence to change VTs

ply-utils: Introduce ply_change_to_vt_with_fd () and ply_change_to_vt ()

When using input devices, set the VT keyboard state with KDSKBMODE so that the VT console doesn't change the LED state

main: Restore terminal line discipline on hide splash

At the moment if a program calls `plymouth hide-splash`
plymouth keeps the terminal in raw mode. That is wrong,
because programs call `plymouth hide-splash` specifically
so they can use the terminal.

This commit makes plymouth restore the terminal to cooked
mode so it's ready for the next program.

Merge branch 'improve-console-viewer-api' into 'main'

ply-console-viewer: Improve API

See merge request plymouth/plymouth!260

ply-console-viewer: Improve API

At the moment injecting data into a console viewer involves calling
ply_console_viewer_parse_lines.

This name isn't optimal, since the data getting injected might not
be a full line at all.

This commit renames it to ply_console_viewer_write, and also adds
a ply_console_viewer_print that adds a printf like function for
convenience.

Merge branch 'terminalwidth' into 'main'

ply-terminal-emulator: Take a maximum_column_count argument that is used in ply_rich_text.

See merge request plymouth/plymouth!256

ply-utils: rename ply_utf8_string_iterator_init to ply_utf8_string_iterator_initialize