Merge branch 'xkbcommon-dep' into 'main'

ply-splash-core: Add xkbcommon dep to pkgconfig file

See merge request plymouth/plymouth!316

ply-splash-core: Add xkbcommon dep to pkgconfig file

This addresses a build blip.

https://gitlab.freedesktop.org/plymouth/plymouth/-/issues/251

Merge branch 'weblate-plymouth-master' into 'main'

Translations update from Fedora Weblate

See merge request plymouth/plymouth!312

Translated using Weblate (Kazakh)

Currently translated at 100.0% (7 of 7 strings)

Added translation using Weblate (Kazakh)

Co-authored-by: Baurzhan Muftakhidinov <baurthefirst@gmail.com>
Translate-URL: https://translate.fedoraproject.org/projects/plymouth/master/kk/
Translation: plymouth/main

Translated using Weblate (Kabyle)

Currently translated at 57.1% (4 of 7 strings)

Added translation using Weblate (Kabyle)

Co-authored-by: ButterflyOfFire <butterflyoffire@protonmail.com>
Translate-URL: https://translate.fedoraproject.org/projects/plymouth/master/kab/
Translation: plymouth/main

Merge branch 'scriptparsekmsg' into 'main'

script: Get the messages from the console and ply-kmsg-reader, and use...

See merge request plymouth/plymouth!277

script: Get the messages from the console and ply-kmsg-reader, and use ply-terminal-emulator and ply-console-viewer to the display messages

Merge branch 'fix-no-unused-warning' into 'main'

ply-buffer: Fix unused-value warning

Closes #246

See merge request plymouth/plymouth!309

ply-buffer: Fix unused-value warning

We currently get warnings during the build like

```
../src/libply/ply-buffer.h:60:20: warning: value computed is not
used [-Wunused-value]
60 |              !_ran && (*bytes = (char *) ply_buffer_get_bytes(buffer),
   |                    ^~
```

This commit changes the macro to use a GCC statement expression with
an if statement to work around the warning.

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

Merge branch 'bias-scale-downward' into 'main'

ply-utils: Only choose scale 2 when the perfect scale would be >= 1.75

See merge request plymouth/plymouth!308

ply-utils: Only choose scale 2 when the perfect scale would be >= 1.75

This is the intended design documented in:
https://gitlab.gnome.org/GNOME/mutter/-/commit/d03dce43786d

And discussed in:
https://gitlab.gnome.org/GNOME/mutter/-/merge_requests/3616

Merge branch 'match-mutter-default-scale' into 'main'

ply-utils: Match mutter's default device scale choice

See merge request plymouth/plymouth!307

ply-utils: Match mutter's default device scale choice

Until now, laptops with a DPI between 192 and 202 would be given a
default scale of 2 by Plymouth, and 1 by Mutter for the login screen.
That made the visual transition a bit ugly so let's match Mutter's
default scale selection. This means the threshold for laptops is now
1.5 x 135 = 202 DPI instead of 192 DPI. And for desktop monitors it's
now 1.5 x 110 = 165 DPI instead of 192 DPI.

Closes: https://bugs.launchpad.net/bugs/2054769

Merge branch 'weblate-plymouth-master' into 'main'

Translations update from Fedora Weblate

See merge request plymouth/plymouth!306

Translated using Weblate (Turkish)

Currently translated at 100.0% (5 of 5 strings)

Co-authored-by: Sabri Ünal <libreajans@gmail.com>
Translate-URL: https://translate.fedoraproject.org/projects/plymouth/master/tr/
Translation: plymouth/main

Merge branch 'fix240' into 'main'

ply-boot-splash: Set unbuffered input when creating a text display

Closes #240

See merge request plymouth/plymouth!303

ply-boot-splash: Set unbuffered input when creating a text display

Merge branch 'ev_led_not_needed' into 'main'

xkb_state_led_name_is_active does not appear to require devices to have EV_LED

See merge request plymouth/plymouth!305

xkb_state_led_name_is_active does not appear to require devices to have EV_LED

This allows the on-screen caps lock indicator to work on HyperV keyboards, and HyperV has one keyboard

Merge branch 'fix245' into 'main'

renderers: Do not assume all keyboards have LEDs

Closes #245

See merge request plymouth/plymouth!304

renderers: Do not assume all keyboards have LEDs

This is an attempt to fix #245

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

label-freetype: fix fallback not working when fc-match isn't available

Closes #239

See merge request plymouth/plymouth!302

label-freetype: fix fallback not working when fc-match isn't available

The new font loading functions introduced in
544e62ac41a490f04d8e6b4e85f8b9fa1171b0cc assume that popen() returns
NULL when fc-match is unavailable or fails. This is incorrect, since
popen() will always start a shell to run the passed command and return a
stream to that shell's stdin and stdout.

This results in an non-null but empty font name being passed to
FT_New_Face(), which fails.

This commit fixes this by also using the fallback font when the font
path read from the popen() stream is empty.

Fixes #239
Fixes regression caused by 544e62ac41a490f04d8e6b4e85f8b9fa1171b0cc

Merge branch 'inibash' into 'main'

plymouth-set-default-theme: Use a more targeted bash ini-parser function...

Closes #238

See merge request plymouth/plymouth!300

plymouth-set-default-theme: Use a more targeted bash ini-parser function instead of sed to handle the config file

Merge branch 'fixinigroup' into 'main'

splash plugins: Fix wrong group being used for console viewer settings in...

See merge request plymouth/plymouth!301

splash plugins: Fix wrong group being used for console viewer settings in space-flares and fade-throbber

Merge branch 'fixinputwarnings' into 'main'

ply-input-device: Fix 'may be used uninitialized' warnings

See merge request plymouth/plymouth!299

ply-input-device: Fix 'may be used uninitialized' warnings

If libevdev gave plymouth an event with an out of range key event type then plymouth would end up using uninitialized variables.

This commit fixes the compiler warning introduced from that unlikely scenario.

Merge branch 'fixvtlesscrash' into 'main'

ply-terminal: Fix crash on vt-less kernels when checking the enabled state

See merge request plymouth/plymouth!298

renderers: Only call ply_terminal_set_unbuffered_input when there is a terminal

ply-terminal: Fix crash on vt-less kernels when checking the enabled state

Merge branch 'fix-hidpi-on-freetype' into 'main'

label-freetype: Fix rowstride bug with hidpi displays

See merge request plymouth/plymouth!297

label-freetype: Force resize calculation when moving control

When moving the label around we need to do a full recalculation
of the metrics, because the position is part of the computation.

label-freetype: Fix rowstride bug with hidpi displays

The freetype plugin correctly doubles the DPI on hidpi displays,
but fails to account for the doubled pixels in display's pixel buffer.

This commit adds a factor of 2 to the size and positioncomputations,
to hopefully fix a row stride and a positioning bug.

Merge branch 'weblate-plymouth-master' into 'main'

Translations update from Fedora Weblate

See merge request plymouth/plymouth!234

Translated using Weblate (Welsh)

Currently translated at 100.0% (7 of 7 strings)

Added translation using Weblate (Welsh)

Co-authored-by: Diogyn D <diogyn@tutanota.com>
Translate-URL: https://translate.fedoraproject.org/projects/plymouth/master/cy/
Translation: plymouth/main

Translated using Weblate (Galician)

Currently translated at 100.0% (7 of 7 strings)

Added translation using Weblate (Galician)

Co-authored-by: Fran Diéguez <frandieguez@gnome.org>
Translate-URL: https://translate.fedoraproject.org/projects/plymouth/master/gl/
Translation: plymouth/main

Translated using Weblate (Dutch)

Currently translated at 100.0% (7 of 7 strings)

Co-authored-by: Philip Goto <philip.goto@gmail.com>
Translate-URL: https://translate.fedoraproject.org/projects/plymouth/master/nl/
Translation: plymouth/main

Translated using Weblate (Czech)

Currently translated at 100.0% (7 of 7 strings)

Co-authored-by: Pavel Borecki <pavel.borecki@gmail.com>
Translate-URL: https://translate.fedoraproject.org/projects/plymouth/master/cs/
Translation: plymouth/main

Merge branch 'fix-freetype-loading-bug' into 'main'

label-freetype: Rework font loading

See merge request plymouth/plymouth!296

label-freetype: Rework font loading

There's currently this function, set_font_with_fallback, that
almost always gets called with a NULL first argument in the
initramfs, forcing the _with_fallback part of the function to
run.

It's a little strange to have a function with a chunk of code that
hardly ever runs. Furthermore there's a bug where the error variable
is left uninitialized in this case leading to the freetype plugin
sporadically failing to load the fallback font.

This commit reworks things to drop set_font_with_fallback, and just
call FT_New_Face directly in the caller. Fallbacks are handled
at the point where the font path is determined
(previously called query_fc_match, now called find_default_font_path.

Merge branch 'fix-pango-hex-box-removal' into 'main'

label-pango: Try to fix up remove_hexboxes_from_pango_layout

See merge request plymouth/plymouth!295

label-pango: Try to fix up remove_hexboxes_from_pango_layout

There are a few issues with the remove_hexboxes_from_pango_layout
function, where it's getting offsets wrong and things like that.

This commit changes the approach entirely. Rather than modifying
the text to remove characters without font converage, it just
changes out the glyphs for those characters to be white boxes.

ply-utils: Make ply_utf8_character_get_byte_type detect invalid bytes

Somehow I neglected to make ply_utf8_character_get_byte_type
differentiate between valid and invalid utf-8.

This commit fixes that.

Merge branch 'dont-set-keyboard-mode-so-much' into 'main'

ply-terminal: Only set keyboard mode when changing it

See merge request plymouth/plymouth!294

ply-terminal: Only set keyboard mode when changing it

Setting the console keyboard mode is not entirely idempotent.
The kernel may flush the input buffer leading to lost
key strokes. We currently set the mode explicitly on graphics
updates as part of our more general "fix things up in case
something during boot screws with the terminal settings" code.

That leads to keystrokes getting eaten.

This commit makes the terminal more careful about setting the
keyboard mode. It now only changes it when plymouth thinks its
necessary to do so.

In the future we could query the keyboard mode and reset it
when it's wrong, but I think we should hold off on doing that
until shown we need to.

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