main: fix kernel command line parsing

check_for_consoles has another bug (surprised?) where it would
jump too many characters forward if the command line has
console=tty0 in it, since tty0 is transparently changed to /dev/tty1.

main: plug fd leak

This plugs an fd leak in an error path

utils: drop ply_list_directory

It's unused code

utils: plug dir fd leak

This commit calls closedir() in an error path.

logger: plug fd leak

This plugs an fd leak in an error path.

text-progress-bar: plug fd leak

This plugs a fd leak in an error path

client: fix fd leak

This plugs an fd leak in an error path

README: resync from wiki

walters pointed out the README had some stale information in
it.  I've copy and pasted the wiki text in, now, so there
will be hopefully a small quantity of said stale information.

autogen.sh: rework to be more buildsystem friendly

One of the goals laid out in
http://people.gnome.org/~walters/docs/build-api.txt
is to get projects to have autogen.sh scripts that
don't invoke configure after generating it.

This commit does that along with some other small
modifications designed to help be more confromant
with the Build API referenced in the above url.

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

event-loop: use static buffer instead of stack allocation buffer

alloca() isn't a good idea to use, in general, because it provides
undefined behavior when it fails.

This commit changes the epoll event buffer from being stack allocated
to static to avoid any potentially problems resulting from stack
space exhaustion.

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

README: spelling fix.

This changes the mispelled word analagous to the
word analogous.

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

throbgress: add plugin decelartion

This should fix the build.

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

throbgress: make colors configurable

This commit makes the background start and
background end colors configurable in throbgress.

Revert "Add --nowait option."

This reverts commit c926bccd83ed1391df80adfecd1229b0327e1f11.

It was an in-progress patch that I carelessly and erroneously
pushed.

x11: translate enter to new line

commit 4081bd29fb471a93daba746643419d858f06bdfe changed
our keyboard handling code to treat newline instead of
carriage return as the enter key.

That commit fixed up the tty settings to work under these
new assumptions.  It failed to update the X11 code, though.

This commit fixes the X11 renderer plugin so it works in
the same way.

Add --nowait option.

With --nowait option, plymouth will not wait for reply from plymouthd.

progress-animation: allow 0 frame animations

Currently ply-progress-animation crashes if you try to pass it
an animation with 0 available frames.

This commit makes it silently succeed.

two-step plugin: free plugin->animation_dir when destroy

key-file: minor code shuffle

Move things around to avoid false positives in coverity.

main: fix console=/dev/tty special case

commit 1d09ad7d51e803e56b52c4e84dd558a7ac4d2e39 contained a mistake
where it checked for "tty" twice instead of "tty" once and "/dev/tty"
once.

This commit fixes that error.

main: special case console=tty

Some people use console=tty in the same way they'd expect
console=tty0 to work.

This commit makes them behave the same way.

keyboard: handle kernel terminal EIO better

In commit 89096d735f78fab300248962030d6554f9894011 we added
reopen retries when the kernel returns EIO to ply-terminal.c.

This is because when the kernel is closing a tty down, that
tty is unavailable to userspace to reopen.

Unfortunately, that commit neglected to inform the ply-keyboard
part of the code when the terminal retry was successful.  The
upshot of this, is that if plymouthd needs to retry opening the
tty, then the splash screens lose control over the keyboard.

This commit changes how input notification is sent to the keyboard
handling code, so the tty disconnects are transparent.

main: allow log output to be redirected

Sometimes putting debug output on the screen has strange side
effects.  This commit adds a new kernel command line option,
e.g:

plymouth.debug=stream:/dev/null

that moves the output "out of the way"

main: write pid file and daemonize last

These actions serve to notify plymouthd's readiness.
We should make sure they are run after plymouthd is actually
done starting up.

main: write pid file even when not daemonizing

This commit moves the pid file writing code to main
from ply_create_daemon, so that it gets run even when we
plymouthd isn't daemonized.

This is more symmetrical, anyway, since unlinking of the pid
file is handled in main.

text-progress-bar: Add support for /etc/os-release

Systemd uses '/etc/os-release' as release file, see
http://0pointer.de/public/systemd-man/os-release.html

main: force frame-buffer plugin at shutdown

This way we hopefully won't get kicked out of KD_GRAPHICS
mode when we're SIGKILLed causing the splash to disappear
prematurely.

main: fix ply_mode_t/ply_boot_splash_mode_t confusion

They're very similar enums and I was using the wrong one,
which caused a harmless compiler warning.

renderer: provide way to override renderer plugin

This will allow us to force e.g. frame-buffer even
if drm would work.

main: create runtime directory if it doesn't exist already

This makes sure there is a convenient place to put pid files.

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.