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

[drm] plug driver_name leak

The previous commit accidentially added a small memory leak.
This commit mops that up.

[drm] Add preliminary support for libkms

This commit adds most of the pieces in place to use libkms, a
library by Jakob Bornecrantz, that abstracts the drm drivers
behind a common api.

Right now, we only fallback to libkms if the existing
backends won't work for the configured hardware.

In theory, this will give us pretty boot in virtual
machines, since libkms has support for the vmwgfx drm driver.

Aside from vmwgfx, libkms also supports intel and nouveau right
now.  When it supports radeon, too, I'll probably switch to
using libkms by default instead of as a fallback.  Eventually,
I'd like to drop all the non-libkms backend bits and the whole
driver vtable abstraction thing from plymouth completely.

This commit is just a copy-and-paste of one of the existing
drm backend files, with changes made to accomodate the libkms
api.  I haven't actually tested it, yet, so it will probably
need changes after I get a chance to do that.

[client] Update top line of help output

It said "Boot splash control client" before, but we do
boot and shutdown splashes.

[main] Update top line of help output

It said "Boot splash control server" before, but we do
boot and shutdown splashes, and the word "control" doesn't
make sense.

[two-step] Add prototype for exported function

[space-flares] mark functions static

[space-flares] Add prototype for exported function

Add prototype for exported function

[fade-throbber] Add prototype for exported function

[throbgress] add prototype for exported function

[label] add prototype for exported function

[label] remove bogus whitespace

[label] mark functions static

[throbber] drop unused variable

[throbber] fix incorrect cast

[animation] drop unsed variable

[animation] fix signedness warnings

[throbber] fix signedness warning

[region] clean up bogus whitespace

[region] drop unused variables

[region] mark do_test static

[rectangle] Remove bogus whitespace

[rectangle] drop unused variable

[region] Use correct type for loops

We were using ints when dealing with rectangles which
use longs.

[two-step] Don't crash if throbber is unavailable

While the code took some steps to make the throbber optional,
it missed conditionalizing usage of the throbber in the
exit path.