Merge branch 'dont-uncrustify-on-debian' into 'main'

ci: Don't run uncrustify on debian

See merge request plymouth/plymouth!196

ci: Don't run uncrustify on debian

The uncrustify is too old for the config we ship.

Merge branch 'move-ci-to-debian-unstable' into 'main'

ci: Update to debian unstable

See merge request plymouth/plymouth!195

ci: Update to debian unstable

It seems the uncrustify in debian stable is too old. Hopefully,
debian unstable has a new enough version.

Merge branch 'kate/WIP/secureboot-check' into 'main'

plugins: splash: two-step: Secure boot check and a warning image

See merge request plymouth/plymouth!176

plugins: splash: two-step: Secure boot check and a warning image

Secure boot is used against several security threats when malware tries to
infect the firmware of the system. Users may inadvertently disable or
software may intentionally disable the secure boot. Consequently, the
system is running on an insecure platform with incorrect configuration. If
Plymouth could offer a warning to the user, the user could reboot and
reconfigure their system or asks for help immediately.

This work can be used to check the secure boot configuration and put a red
warning image on the screen if the secure boot is disabled. Also, this
check can be utterly disabled through the kernel parameter for testing.
If the parameter "secure_boot.warn_if_disabled=false" appears in the
kernel parameter, the secure boot check will be disabled.

Signed-off-by: Kate Hsuan <hpa@redhat.com>

libply: ply-utils: Add ply_is_secure_boot_enabled () helper

This checks the secure boot status. If the secure boot are enabled,
return true. Otherwise, return false.

The system secure boot settings is at
/sys/firmware/efi/efivars/SecureBoot* and the fifth byte should be
0x1 which means the system secure boot is enabled.

Signed-off-by: Kate Hsuan <hpa@redhat.com>

Merge branch 'move-ci-to-f37' into 'main'

ci: Update to fedora 37

See merge request plymouth/plymouth!185

ci: Update to fedora 37

It seems like uncrustify is changed slightly incompatibly between
35 and 37, so just move to 37.

scripts: Add -B to interdiff in check-format

It's currently complaining about an extra blank line in each file.
That's a little extreme.  Add -B.

ply-utils: Drop linux/fs.h include

It was needed long ago for a function we no longer even have.

Now it's causing compile errors on Fedora 37 because it's conflicting
with sys/mount.h.

This commit drops it.

Merge branch 'drop-vim-lines' into 'main'

src: Drop vim: lines

See merge request plymouth/plymouth!184

src: Drop vim: lines

Many of the source files in tree have "vim:" lines to tell vim a
hint about our coding style.

Unfortunately, they were written when plymouth used the GNU coding
style, and it, of course, doesn't anymore.

This commit just drops them since the default vim indentation options
match pretty well with the plymouth coding style now, anyway.

Merge branch 'update-uncrustify-config' into 'main'

scripts: Update uncrustify config

See merge request plymouth/plymouth!183

scripts: Update uncrustify config

It looks like I forgot to merge the latest uncrustify config last
time I did an uncrustify run.

This commit puts it in place.

Merge branch 'diff-less' into 'main'

scripts: Diff less

See merge request plymouth/plymouth!182

scripts: Diff less

I think check-format.sh might get into trouble in some cases when
a non source file gets changed.

This commit makes the diffing more targeted

Merge branch 'cut-the-crust' into 'main'

src: Run through uncrustify

See merge request plymouth/plymouth!181

src: Run through uncrustify

I was going to fix the preexisting style issues piecewise as they're
changed, but it's actually making reviewing patchsets noisy.

This commit just bites the bullet and does a full run through
uncrustify.

Merge branch 'runtimedir-fix' into 'main'

Make use of standard --runstatedir flag vs custom --with-runtimedir

See merge request plymouth/plymouth!178

Fix daemon install ignoring configured runstatedir

Use standard runstatedir vs custom flag

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

client: Free command parser after event loop

Closes #175

See merge request plymouth/plymouth!175

script: Don't crash for scripts without input validation functions

commit 5d1b49c601a085120e834c1e9134593318465eec adds support for
filtering input to plymouthd and the script plugin.

The script plugin additions, unfortunately, fail to properly handle
scripts that don't add a validation function.

This commit fixes that by assuming no validation function means
no filtering.

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

command-parser: Don't add duplicate command to command list

ply_command_parser_get_options inexplicably adds a duplicate
command to the list of available subcommands every time it's
called. This must be some sort of cut and paste bug, it shouldn't
be modifying the list.

That leads to double frees when freeing the command parser.

This commit fixes that.

Related: https://gitlab.freedesktop.org/plymouth/plymouth/-/issues/175

client: Free command parser after event loop

commit ed7b1690f0054888795059c2ef6ddd2f8107786a added an explicit
command parser free call on exit to make memory analysis tools
give better results.

That commit freed the command parser a little too early, though,
before the event loop gets freed and the event loop exit handlers
are run.

The command parser uses an exit handler to clear a weak reference
on the event loop, so freeing the command parser before the event
loop can lead to crash.

This commit moves the free call a little lower.

Related: https://gitlab.freedesktop.org/plymouth/plymouth/-/issues/175

Merge branch 'wip/uncrustify-updates' into 'main'

.gitlab-ci: Don't regress formatting

See merge request plymouth/plymouth!174

.gitlab-ci: Don't regress formatting

The formatting in the codebase isn't perfect, but there's no
reason to make it worse.

This commit adds a CI check to make sure new code is styled
right.

.gitlab-ci: Make it work better on merge requests

gitlab CI apparently exposes more useful environment variables
if `only: merge_requests` is put on each job, so do that.

.gitlab-ci: Update Fedora version to 35

30 is ancient at this point.

script: Enforce separate lines for function parameters

The moment it's possible to sneak in functions like:

static void
foo (int param1, int param2)
{
}

when the style requires parameters on separate lines.

This updates the uncrustify config to account for that.

Merge branch 'check-shell-before-merge' into 'main'

.gitlab-ci: Check for shell errors in CI

See merge request plymouth/plymouth!172

.gitlab-ci: Check for shell errors in CI

Recently a shell error snuck into CI and I didn't notice it because
it was in the tail end of a short-circuited conditional.

This commit adds a little shell validation to CI to avoid that
sort of thing in the future.

Merge branch 'plymouth-set-default-theme' into 'main'

scripts: plymouth-set-default-theme reporting missing right brace

See merge request plymouth/plymouth!171

scripts: plymouth-set-default-theme reporting missing right brace

Fixes issue:

/usr/sbin/plymouth-set-default-theme: line 199: [: missing `]'

when executed as plymouth-set-default-theme spinner

Merge branch 'tt-keyfilter' into 'main'

core, script: add support for filtering input characters in plugins

See merge request plymouth/plymouth!169

core, script: add support for validating input characters in plugins

This adds also '\e' to be parsed as escape character in script strings.

Together these allow possibility to create scripts that allow only
numeric PINs, and prevent switching to text mode.

Merge branch 'set-default-theme-improvements' into 'main'

Improve plymouth-set-default-theme a bit

Closes #174

See merge request plymouth/plymouth!167

scripts: Support -R with -r and standalone for plymouth-set-default-theme

Right now plymouth-set-default-theme only supports rebuilding the initrd if a theme
is passed explicitly.

This commit improves the script to also work for --reset and even in a
standalone mode where no theme is passed at all.

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

scripts: Support short options being squished togethers for plymouth-set-default-theme

People expect to be able to squish short options together, so
this commit adds some glob matching to support that feature.

Merge branch 'mem-leak' into 'main'

client: fix memory leak state.client and state.loop

See merge request plymouth/plymouth!170

event-loop: change assert to just check for is_running

Simlify assert in ply_event_loop_free, to check that the event loop is
not running.

command-parser: add more free's

A couple of resources not free'd

client: call cleanup functions everytime we return from main

We were not cleaning up allocated resources in many of the code
execution paths in main()

Merge branch 'script-password-argument' into 'main'

script: add password argument to display password callback

Closes #150

See merge request plymouth/plymouth!166

script: add String.Length function

Add String.Length function which returns the length of a string.
It will make it possible to get the number of bullets from the
display prompt callback inside scripts.

Add display_prompt function

Add display_prompt function which will be called for both passwords
and questions. This will make it possible to get a plaintext password
string.
SetDisplayPromptFunction will make it possible to set this callback
from a theme script.

Closes #150

Merge branch 'device-manager-fbdev-add-after-timeout' into 'main'

ply-device-manager: Accept new simpledrm/fbdev devices from udev events after timeout

Closes #167

See merge request plymouth/plymouth!165

ply-device-manager: Accept new simpledrm/fbdev devices from udev events after timeout

Before the device-timeout elapses verify_add_or_change () blocks new
simpledrm and fbdev getting added through udev add events because we
really only want native drm devices.

Then when the timeout triggers the list of udev devices existing at
that point int time is rescanned and any drm (including simpledrm) and
fbdev devices are added (filtering out duplicates).

But what if a simpledrm or fbdev device for some reason only shows
up after the timeout? Before this change those would then be ignored
for ever.

Add a manager->device_timeout_elapsed check and make verify_add_or_change ()
accept any drm + fbdev devices after the timeout. Note this addresses
a mostly theoretically issue since in practice a simpledrm or fbdev device
showing up later is very unlikely.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

ply-device-manager: verify_add_or_change(): Move local_console_is_text check

Move the local_console_is_text check outside of the
"if (subsytem == SUBSYSTEM_DRM)" block.

This check is equally relevant for SUBSYSTEM_FRAME_BUFFER.

Note by itself this is a no-op since verify_add_or_change() *always*
returns false for SUBSYSTEM_FRAME_BUFFER devices.

This is a preparation patch for making verify_add_or_change() not
return false when manager->device_timeout_elapsed is set.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

Merge branch 'simpledrm-enumeration' into 'main'

ply-device-manager: Treat SimpleDRM drm devices as fbdev devices

Closes #167

See merge request plymouth/plymouth!163

ply-device-manager: Treat SimpleDRM drm devices as fbdev devices

Simple-framebuffer devices driven by simpledrm lack information
like panel-rotation info and physical size, causing the splash
to briefly render on its side / without HiDPI scaling, switching
to the correct rendering when the native driver loads.

To avoid this treat simpledrm devices as fbdev devices and only
use them after the timeout.

Also adds 2 exceptions to this:

1. If nomodeset is passed on the kernel commandline then no native
drivers will load, so in this case it is best to immediately use
SimpleDRM devices when they are detected.

2. On some devics the firmware leave the panel black at boot. In this
case it is desirable to show the splash to the user ASAP so that there
is some visual feedback that the device is booting. Add a support for a
"plymouth.use-simpledrm" kernel cmdline option to show the splash
immediately on SimpleDRM devices rather then waiting for the native
driver to load.

Closes #167

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

ply-utils: Reintroduce ply_string_has_prefix helper

ply_string_has_prefix was dropped in commit c7965ea19abf ("ply-utils:
Drop unused ply_string_has_prefix helper"). We have a need for this
helper again, so reintroduce it.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

Merge branch 'misc-cleanups' into 'main'

Misc. cleanups

See merge request plymouth/plymouth!164

ply-device-manager: Remove unnecessary subsystem != NULL check

The ply-device-manager.c already assumes that the return value of
udev_device_get_subsystem () is never NULL in many places, including
in the condition of the "else if" just below the check which is
being removed.

Remove the one lonely check for it being NULL for consistency.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

ply-device-manager: Fix double-assignment of subsystem

The subsystem variable is also initialized with udev_device_get_subsystem ()
a couple of lines lower.

Remove the first initialization, so that udev_device_get_subsystem ()
does not get called unnecessary when the function exits early because
of the action value.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

gitignore: Add plymouthd-fd-escrow to gitignore

Add the new plymouthd-fd-escrow binary to gitignore.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

Merge branch 'main' into 'main'

Add support for CSI sequences

Closes #166

See merge request plymouth/plymouth!162

Add support for CSI sequences

Merge branch 'ignore-clangd-files' into 'main'

Ignore generated files required by clangd

See merge request plymouth/plymouth!160

Ignore generated files required by clangd

Also add .dirstamp which seems to be generated by
autoconf/automake/make.

`compile_commands.json` is also used by standalone `clang-tidy`

Merge branch 'reproducible-build-fix' into 'main'

configure: Generate version from date of last commit not current date

See merge request plymouth/plymouth!158

configure: Generate version from date of last commit not current date

Using the date at build time prevents the build from being reproducible
later.

In order to facilitate reproducible builds, this commit changes the
logic to derive the version from the date of the last commit.

Merge branch 'romanian-translation' into 'main'

Add Romanian translation

See merge request plymouth/plymouth!155

Add Romanian translation

Merge branch 'fix-buffer' into 'main'

drm: do not reset the buffer twice

See merge request plymouth/plymouth!152

drm: Retry setting scanout buffer on failure

Plymouth currently assumes that setting the scanout buffer will succeed.
if it fails because of a driver bug or transient failure it should try
again next frame.

This commit adds that error handling code.

Merge branch 'version-naming-change' into 'main'

autogoo: Determine project version from date

See merge request plymouth/plymouth!153

autogoo: Determine project version from date

Numbers are arbitrary, and we've never successfully made it to
version 1.0 after like 15 years or something.

Furthermore, plymouth has a very slow release schedule at present
and some distros hate building from git.

So, I'd like to start generating tarballs more regularly.

Adopting a version number derived from the date will help facilitate
that.

This commit changes AC_INIT to compute the version automatically.

https://gitlab.freedesktop.org/plymouth/plymouth/-/issues/143
https://gitlab.freedesktop.org/plymouth/plymouth/-/issues/158

Merge branch 'wip/use-logo-file' into 'main'

spinfinity: use logo file passed to configure

See merge request plymouth/plymouth!110

spinfinity: use logo file passed to configure

At the moment we hardcode the header to a logo file that ships
in fedora.

This commit changes it to use the logo configured by the distro.

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

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

Translations update from Weblate

See merge request plymouth/plymouth!146

Translated using Weblate (Belarusian)

Currently translated at 100.0% (7 of 7 strings)

Co-authored-by: Yauhen Bugamol <3abac@3a.by>
Translate-URL: https://translate.fedoraproject.org/projects/plymouth/master/be/
Translation: plymouth/master

Merge branch 'add_croatian_language' into 'main'

Add Croatian language

See merge request plymouth/plymouth!148

Add Croatian language

Merge branch 'fix-configure-help-string' into 'main'

configure: Fix help string for --with-background-end-color-stop

See merge request plymouth/plymouth!149

configure: Fix help string for --with-background-end-color-stop

Looks like it was copied from --with-background-start-color-stop and
never updated to reflect the option it is describing.

Merge branch 'fix-twostep-comments' into 'main'

two-step: Fix typos and missing words in comments

See merge request plymouth/plymouth!150

two-step: Fix typos and missing words in comments

Merge branch 'configurable-show-animation-fraction' into 'main'

two-step: Make SHOW_ANIMATION_FRACTION configurable

See merge request plymouth/plymouth!151

two-step: Make SHOW_ANIMATION_FRACTION configurable

Allow themes to configure at which percentage of the boot progress the
animation should transition to the end animation.

At Endless we use this to only show the end animation from the start of
the boot splash by setting it to 0.0.

Merge branch 'honour-rotation' into 'main'

drm: Honour screen rotation when detecting HiDPI

See merge request plymouth/plymouth!147

drm: Honour screen rotation when detecting HiDPI

Currently if the screen is rotated by 90 degrees, the effective
width/height will be swapped. Thus we will incorrectly detect the panel
as HiDPI.

Signed-off-by: Emil Velikov <emil.velikov@collabora.com>

Merge branch 'themepath-cfg' into 'master'

main: add ThemePath configuration option

See merge request plymouth/plymouth!142

scripts: support populating from configurable theme dir

Support populating initramfs with themes from the configurable theme
directory.

main: add ThemeDir configuration option

Add a new option called ThemeDir to the configuration file, so a
configurable directory can be used to ship themes. The option applies
only to the theme specified in the same configuration file.

main: refactor code for searching the theme path

Refactor code that searched for the theme configuration so it can be
reused.

ply-utils: add macro to count the number of elements in an array

main: free path variables before exiting

Free path variables before exiting so the code is more asan/valgrind
friendly.

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

Translations update from Weblate

See merge request plymouth/plymouth!145

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/master

Merge branch 'crash_on_hide_message' into 'master'

Fix crash when hiding message in details splash mode

See merge request plymouth/plymouth!140

Fix crash when hiding message in details splash mode

Merge branch 'ignore-serial-option' into 'master'

main: add ignore-serial-consoles option

See merge request plymouth/plymouth!143

main: add ignore-serial-consoles option

Add --ignore-serial-consoles option so we can ignore serial consoles
also when starting plymouthd and not only from the kernel command
line.

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

Translations update from Weblate

See merge request plymouth/plymouth!139

Translated using Weblate (Finnish)

Currently translated at 100.0% (7 of 7 strings)

Added translation using Weblate (Finnish)

Co-authored-by: Jan Kuparinen <copper_fin@hotmail.com>
Translate-URL: https://translate.fedoraproject.org/projects/plymouth/master/fi/
Translation: plymouth/master

Translated using Weblate (Sinhala)

Currently translated at 14.2% (1 of 7 strings)

Co-authored-by: Hela Basa <r45xveza@pm.me>
Translate-URL: https://translate.fedoraproject.org/projects/plymouth/master/si/
Translation: plymouth/master

Translated using Weblate (Chinese (Simplified) (zh_CN))

Currently translated at 100.0% (7 of 7 strings)

Co-authored-by: Liu Tao <lyuutau@outlook.com>
Translate-URL: https://translate.fedoraproject.org/projects/plymouth/master/zh_CN/
Translation: plymouth/master

Translated using Weblate (Persian)

Currently translated at 100.0% (7 of 7 strings)

Added translation using Weblate (Persian)

Co-authored-by: Danial Behzadi <dani.behzi@ubuntu.com>
Translate-URL: https://translate.fedoraproject.org/projects/plymouth/master/fa/
Translation: plymouth/master