gdm: don't do legacy gdm transition at shutdown

systemd: change instrumentation points for systemd

We don't want to enable console verbosity until the splash is up,
and we want to disable it as soon as we aren't logging the console
anymore.

configure: s/text/test/

Just a typo from the previous commit

systemd: toggle systemd messages when toggling console redirection

Normally systemd is very mute about messages.  It's important,
though for it to be chatty when plymouth is running so we can
show verbose messages to the user when they hit escape.

This commit adds a new --enable-systemd-integration configure
flag which explicitly tells systemd when to print messages to
the console.

This may get dropped in the future in lieu of init script
changes doing this instead of plymouth directly.

main: consolidate ply_terminal_session_detach calls

It's called in several places through the code. This commit
moves it into a function and just calls that function instead.

client: small leak fix

utils: drop some bizarre looking code

splash: add new spinner theme

This is a very simple theme with a small spinner on a dark background.

main: add new --kernel-command-line option

It's really annoying when you're running plymouthd in
debug mode and don't happen to have "splash" on your kernel
command line and then need to reboot.

This commit adds a new debug option to override the kernel
command line for ths system.

progress-animation: fail load if there are no frames

If there are no progress animation frames we need to
fail from the load function so plugins can deal with it
gracefully..

two-step: make progress animation optional

upstart-bridge: add new helper program

This commit adds a new program, plymouth-upstart-bridge,
the listens for upstart state changes and sends them to plymouth,
or prints them out as appropriate.

client: allow reply handlers to be optional.

This commit lightens the hard requirement that every
client request has to have a function handling the reply.

There's really no reason to be so strict, and it makes
life easier for none /bin/plymouth clients this way.

drm: don't light up unlit monitors

If a monitor is dark when plymouth is started, we shouldn't
try to light it up.  There are rules on which outputs can be
attached to which controllers and we're very likely to break
those rules just assigning things willy-nilly.

The kernel should set us up in the way it thinks is best anyway.
There's no reason to second guess the kernel here.

terminal: Keep retrying to reopen tty on EIO after hang up

If the tty hangs up on us, the kernel will return EIO while it's
closing down.  As far as I know, there's no way to get notified
when it's back up again, so just keep retrying in a loop over
the course of a second or so.  Eventually give up, though.

This is really suboptimal and suggests we should get away from
using ttys for input at some point.

client: do not wait forever if daemon is not started

Currently plymouth --wait will wait forever if daemon is not
started. This does not look right - we were asked to wait for
daemon to stop and daemon is obviously stopped. So make it
exit right away in this case.

This avoids timeouts during systemd boot if plymouth happened to be
stopped (or never started) before plymouth-quit-wait had chance to
start.

Signed-off-by: Andrey Borzenkov <arvidjaar@gmail.com>

terminal: apply terminal settings immediately

utils: don't depend on PAGE_SIZE

It's not defined for all architectures.

We don't really need the page size anyway, we just need a reasonably
large value.

main: convert tab to spaces

tests: don't run during make check

They aren't unit tests so running them in make check is wrong.

Eventually, I'd like them to be unit tests, though, and not
just little scratch programs.

terminal: unlock tty before mucking with it

It seems that sometimes when the tty is stolen from us
it remains locked when we get it back.  That would be okay
if the attributes were still kosher, but they aren't.

This commit unlocks the tty before trying to reinitialize
the terminal with suitable attributes.

Debugged by Andrey Borzenkov <arvidjaar@gmail.com>

client: clean up warnings

drm: clean up warnings

frame-buffer: clean up warnings

two-step: fix warnings

space-flares: clean up warning

text: clean up warning

fade-throbber: clean up warnings

throbgress: clean up warnings

label: clean up warnings

main: detach from terminal session on failure

main: make duplicate plymouthd's a silent error

main: clean up warnings

image: clean up warning

text-progress-bar: clean up warnings

region: clean up some warnings

array: clean up a warning

event-loop: clean up some warnings

animation: add debug spew

two-step: Add more debug spew

boot-server: improve debug output

We've written "could not write bytes" in many different
places.  This changes the error message to be more domain
specific.

boot-server: fix debug output when message is from init

We were always trying to show the parent process name,
but init doesn't have a parent.

This commit special cases init, since it's a special case.

drm: reduce minimum build requirements

This patch adds the respective configure options to make it possible to
disable libdrm_intel, libdrm_radeon, libdrm_nouveau, and libkms
independently from each other.

https://bugs.freedesktop.org/show_bug.cgi?id=29804

pixel-buffer: Fix overflow in blend_two_pixel_values

It was using 16bit types to hold the intermiediate
results of the blend, but for high intensity,
low opacity input values it could need up to
17 bits to prevent overflow.

https://bugs.freedesktop.org/show_bug.cgi?id=33129

main: remove tty0 from list of fallback ttys

It's not a valid fallback tty since writing to
it would create a feedback loop (since it's
redirected with /dev/console)

main: Always translate tty0 to tty1

Another bug in check_for_consoles...

We can't ever write to tty0 directly, because
it is redirected (just like /dev/console).  Previously
we would translate the call to tty1, but commit

c40fd792b6edf931a6bbe2ec23518b57483a4e2f

broke that.  This commit fixes it again.

terminal: don't rely on strlen(bytes) for write size

We're printing stuff to the terminal.  This may include
NUL bytes once in a while.  We shouldn't rely on strlen()
to determine how many bytes to write.

boot-client: fix fallback socket support

Commit 3ec007a4820e53aed6713620173bd997957754ec did not
properly check for success when connecting to the fallback
socket path and instead always failed.

This commit fixes the code to properly check for success and
proceed.

Signed-off-by: Andrey Borzenkov <arvidjaar@gmail.com>

utils: if concrete/abstract socket selection

Commit 9de731ed29db4670c1641aedd945c56fd1a3feed caused
plymouth to erroneously drop the leading '\0' on the old
abstract socket making fallback socket support not work.
As result new clients (after update) can no longer communicate
with old running daemons (such as from the initrd).

This commit restores the leading '\0' for both
abstract sockets, trimmed and non-trimmed.

Commit 3ec007a4820e53aed6713620173bd997957754ec did not
check for success with old socket path and failed
always. Properly check for success and proceed.

Signed-off-by: Andrey Borzenkov <arvidjaar@gmail.com>

main: don't pause on crash

plymouthd would previously pause() when crashing if debug
mode was enabled so that it could be attached to with a debugger.

pausing indefinitely during boot up is often a bad idea though.

This commit changes it to sleep 30 seconds instead.

terminal: Don't set ISTRIP in terminal attributes

In certain error paths plymouth would erroneously set
ISTRIP on the terminal.  This is bogus in the same way
the changes fixed by commit ea394383c54dda771dcf59e7ec11280fa945280b
were bogus.

configure: move with_gdm_autostart_file=no to else clause

doing it unconditionally seems to confuse autogoo.

viewer: always dist desktop file

Even if we aren't installing it, we still want it to get shipped
when doing "make dist"

boot-server: don't print error when client goes away

Right now, anytime the server is unable to respond to a
client it puts an ugly message on the screen:

"could not write bytes: Broken pipe"

or some such.  That message isn't really useful unless you're
debugging your distribution, so change it from a ply_error to
ply_trace.

populate-initrd: pre-expand logofile variable

The logofile by default is $datadir/plymouth.png

$datadir contains a reference to $datarootdir, so
we need to preexpand the variable in configure, for
the right value to get written to plymouth-populate-initrd.

populate-initrd: don't hardcode client and daemon path

When not installing in system root, the populate initrd script would not
install the client or daemon due to the paths being hardcoded.
Environmental variables are also now available to override the defaults.

image: replace deprecated libpng function

The png_set_gray_1_2_4_to_8 function is deprecated and has been removed
from libpng14. Now png_set_expand_gray_1_2_4_to_8 is used instead which
is compatible with libpng-1.2.9 and greater.

viewer: don't build viewer by default

The viewer is useful for seeing boot messages after boot up.
It does this by showing a notification icon in the event there
is a problem during boot.

Notification icons aren't as en vogue as they once were, however.

Ideally, we would have a more structured and semantically aware
way to deal with specific boot problems.

This commit turns the icon off by default.  It can still be built
with a --with-log-viewer

Bug: https://bugs.freedesktop.org/show_bug.cgi?id=30724
Reported By: William Jon McCann <william.jon.mccann@gmail.com>

boot-server: make update handling asyncrhonous

This call needs to be cheap since some distros
do a ton of updates.

utils: use system default max for listen()

There's apparently a define that gives the right value to
pass to listen.  So this commit makes us use that instead of
our arbitrary "32"

list: drop node_set_data method

It's not used anywhere, so drop it.

progress: plug small memory leak

We weren't freeing the message structures.

main: NULL initialize "console" variable

check_for_consoles is a really small function, that has historically
been loaded with bugs.

This commit fixes another bug in it, where the consoles variable is
never initialized to NULL.

branch-merge: clean up socket name

The plymouth client talks to the plymouth daemon via
an abstract socket.  There aren't a lot of "rules" for
the name of the abstract socket, and so plymouth has just
used "/ply-boot-protocol" for the name.  While this is
perfectly valid, a number programs (hal, udev, etc) use
the convention "/org/freedesktop/program".

"/org/freedesktop/plymouthd" is certainly much nicer than
"/ply-boot-protocol" so we're going to change to that, and
fall back to the old name for compatibility.

One other niggle is trailing zeros in the name.  The socket
address is stored in a fixed size buffer.  Traditionally,
programms would pass the size of the entire socket structure,
including the full size of the aaddress buffer to bind and
connect.  This means that any NUL bytes in the address buffer
after the address become part of the address.  This means users
looking at /proc/net/unix will see all the extra NUL bytes.

One trick that some programs employee to skirt around this problem,
is to pass only the size of the structure less the trailing
NUL bytes of the address buffer to bind and connected.  While maybe
not 100% kosher, this works okay in practice.

plymouth will now use that trick as well.

protocol: change socket path

Before we were using

  /ply-boot-protocol\0\0\0\0...\0

for our address which really uglifies /proc/net/unix
and doesn't match what a lot of other programs do.

This commit changes the address to just

  /org/freedesktop/plymouthd

which is much nicer.

protocol: drop leading \0 from socket path

The socket path is currently defined to be:

 #define PLY_BOOT_PROTOCOL_SOCKET_PATH "\0/ply-boot-protocol"

The \0 is because it's an abstract socket, and abstract sockets
have a leading NUL.  The code always ignores the NUL though and
adds it back later, so it's not needed.

This commit just drops it.

utils: add support for "trimmed abstract" sockets

Right now plymouth listens on an abstract socket with a name
that has a bunch of trailing NUL bytes.  These zeroes uglify
the output of /proc/net/unix among other things.

This commit adds support for a new "trimmed abstract" socket
type, which drops the zeroes.

A subsequent commit will actually change plymouthd to use the
new api.

set-default-theme: redirect to /dev/null not /null

The set-default-theme script was incorrectly redirecting
stderr from a grep command to /null instead of /dev/null.

This causes a /null file to get created on the filesystem.

Bug: https://bugs.freedesktop.org/show_bug.cgi?id=30576
Reported by: David LeBlanc <leblancdw@yahoo.com>

viewer: Add a title to the status icon

set-default-theme: exit 0 at bottom of file

This is so that if there is no --rebuild-initrd it still
exits successfully.

terminal-session: add debug messages

terminal-session: try to continue logging after tty disconnect

Currently, we give up on boot logging after a tty disconnect.
This commit makes the terminal session reattach to the tty, and
continue logging.

terminal-session: rename "session_done" to "session_hangup"

In truth, we can get hangups at any time during boot.  It doesn't
mean the session is done, so use better terminology.

main: add __DATE__ to debug prolog

When looking at debug logs, it's often useful to know
how old the build of plymouth featured in the log is.

main: add plymouthd: prefix to some error messages

This will make it clear the error messages are coming
from plymouthd and not some other component during boot.

main: use ply_error_without_new_line for help string

It already has new lines in it.

utils: Improve debug spew in ply_create_daemon

Right now if plymouthd dies while daemonizing, we show a horrible
error message on the console:

could not read byte from child: Success

This commit mops that up, so we give a little clue why plymouthd's
child process died in its infancy.

main: Fix up check_for_consoles

commit c40fd792b6edf931a6bbe2ec23518b57483a4e2f was just wrong.

Attempt to fix it up to be less awful.

utils: plug memory leak

commit ffdfb78be1bcda708fd89faf645b582f88c6f1d5 introduced
a memory leak because of a copy-and-paste-o.

boot-server: print who is making requests

One of the big painpoints in plymouth distro integration
out which distro scripts are calling into plymouthd.

This commit makes plymouthd output that information whenever
there is a request from a connected client.

gitignore: Ignore generated script string files

script: Add compatibility redirect for set message function

client: Alias display-message command to message for backward compatibility

command-parser: Allow commands to be aliased

Adds aliases to commands which allow backward compatibility to old names.

script: Implement hide message function in the example script theme

This leaves ugly spaces where messages used to be, but is simple.

script: Add hide message support to the script plugin themes

This also renames SetMessageFunction to SetDisplayMessageFunction, and adds
SetHideMessageFunction

main: Inform plugins of hidden messages

Informs plugins of each message that is now hidden. Later could match on
wild-cards, but plugins will get a list of exact string matches.

boot-splash: Add "hide message" splash plugin call

No splashes use this yet.

boot-splash: Clean up indenting and function order

Display normal, password, quastion should be together

boot-splash: Clean up indenting

main: Remove the hidden message from the message list

The plugins are not updated unless they are switched off and on again

boot-client: Add hide-message client command

This renames "message" command to "display-message".

boot-server: Add "hide message" handler to boot server

Main server currently sets this to NULL to disable this callback.

protocol: Add "hide message" command to the protocol

This also renames the "message" command to "show message".

details: Implement display_message

Messages are queued until any question or password entry prompts
complete.

https://bugs.freedesktop.org/show_bug.cgi?id=29035

main: Open /proc/cmdline by absolute path

Open /proc/cmdline, not proc/cmdline.  (Technically this doesn't matter
in the daemon, since it's already done chdir ("/"), but the client does
need this and it's clearer to have them match.)

client: Open /proc/cmdline by absolute path

Open /proc/cmdline, not proc/cmdline.

[script] Add font selection argument to text to image capability

Enables scripts to choose the font they want with a sixth argument to
Image.Text API:
new_image = Image.Text("Hello", 1, 1, 1, 1, "DejaVu Bold,Italic 18");

[label] Add font controls to label plugins

[label] Factorize some font init code