[throbber] set is_stopped after stopping

The throbber variable has an is_stopped state
variable that decides whether or not draws happen.

It starts off false, and gets toggled when started.

Unfortunately, it never gets reset to false
after be stopped.  This commit fixes that.

Handle NULL boot_buffer

boot_buffer may be NULL if plymouthd failed to attach to the console
session (e.g. if booting without an initramfs so /dev/pts isn't mounted
yet).  Handle this gracefully rather than segfaulting.

Handle plymouth:splash or plymouth:debug as last argument

If plymouth:splash or plymouth:debug is the last argument in
/proc/cmdline, then it will be terminated with '\n' rather than ' '.
Handle this.

Fix parsing of plymouth:debug=file:

Short-circuit evaluation meant that the path variable was never set if
plymouth:debug=file: was used.  Switch the ordering around to fix this.

[terminal] wait for vt switching to finish on quit

After changing the active vt during start up,
we get a signal when it finishes, because we're in
VT_PROCESS mode, so we don't need to explicitly block
waiting for it to finish.

During the quit path, though, we aren't in VT_PROCESS mode anymore,
won't get any signals, and still need to know that the VT switch
is finished before informing the client that the quit is
finished.

This commit forces ply_terminal_deactivate_vt to block until the
VT switch it initiates finishes.

[terminal] Add some debug spew

[splash] export config and policy directories

Export configuration and policy directories used by plymouth.

[main] do not set state->keyboard twice

state->keyboard is set already in set_keyboard, there is no need to do
the work twice.

[script] Remove debugging trace line

[script] Allow passing of values from the .plymouth file to the script

An example in the script.plymouth shows a value being set. These values are
then present at the first execution of the script set in the global scope.

[key-file] Add foreach function to access all stored values

Supplying a callback function, you can get each group/key/value triple. Allows
accessing values in the key file for which you may not know the keys.

[script] Default to text color alpha of 1 if nothing was passed

If NULL or no variable was passed, the aplha variable would be read as NaN
which gets clamped to 0 so the text would be completely transparent.

[main] Create a list of messages and when showing splash display them

The system now keeps a list of messages so they can be shown again when the
user flips themes or if the messages were passed before the splash was shown.

This also enables the splashes to show multiple messages (should they choose to
do so). There will later be a way of undisplaying a message.

[progress-animation] Typo bug in calculating max height

Was looking at the width rather than the height of the second image

[progress-animation] Remove unnecessary draw area

Must have been left over from some debugging

[progress-animation] Remove some now unnecessary area calculations

[progress-animation] Convert more functions to use buffer fill with buffer

[image] Give get buffer function more sensible name

There was a "_convert_" in the middle of the function name from a clumsy copy
paste.

[progress-animation] Use buffer fill with buffer instead of with raw data

[entry] Allow setting of text color

Default color is black.

[entry] Fix ask question entry text

The text was not being shown because show was not called. Also the default
colour is white which was not showing up on white background.

[entry] Convert from drawing image raw data to using pixel buffers

[image] Add asserts to make sure NULL is not passed as the image

[image] Allow access to the internal image buffer

Adds ply_image_convert_get_buffer to get the image buffer without destroying
the image. Forgot to add the .c.

[image] Allow access to the internal image buffer

Adds ply_image_convert_get_buffer to get the image buffer without destroying
the image.

[script] Simplify sprite area calculation

The sprite area no longer needs to be calculated as the buffer holds that
information.

[script] Use fill with buffer function

[animation] Use fill with buffer function

[throbber] Use fill with buffer function

[pixel-buffer] Add fill buffer with buffer functions

These draw the content of one buffer into another. This is cleaner than using
the raw data and passing the width and height of the data separately.

[animation] Move frames from image to pixel-buffers

The frames are now stored as pixel-buffers which will be cleaner to draw from
in the future.

[throbber] Move frames from image to pixel-buffers

The frames are now stored as pixel-buffers which will be cleaner to draw from
in the future.

[pixel-buffer] Add get width and height functions

These were accessible by the get_size function, but individual functions are
more convenient.

[main] Ignore keyboard changes when deactivating

There is a small window after plymouth is told
to quit or deactivate before it actually does.

During this window, if the user happens to hit
escape, bad things could happen.

We really don't want to mess with, e.g., the tty
settings when we're about to exit.

This commit puts plymouth in a sort of "degraded"
mode while it's deactivating.  During this time
frame, user input is ignored.

[terminal] Lock terminal settings

From time to time, various external programs
will muck with the tty we're using and make
the users password for encrypted disks show
up, make the enter key not work, etc.

We used to work around this by resetting the
tty the way we like it everytime we write the
screen.

We no longer do that after commit

e9a22723da7c9400d25aeff2625651b3d03be43f

Instead of changing it every time, it's probably
better to just prevent other programs from messing
up the settings in the first place.

This commit locks the terminal so if those programs
try to change the settings, they fail.

A better long term solution might be to get user input
/dev/input instead of the tty

[pixel-buffer] Remove offsets from buffet fill with data functions

The x and y offsets were supposed to allow only a part of the image data to be
drawn, but this didn't work correctly. Instead the clip region can be used to
emulate the same effect. So the x and y paramiters in these functions are now
dropped. The only instance that used these now uses a clip. All other calls had
x and y set to zero.

[event-loop] Add reference count to event sources

This prevents the event loop from freeing sources
early in an iteration of the loop, and then dispatching
handlers for the source later in that same iteration.

[drm] Drop drm/ from include directives

One file was missing the fix.

[configure] bump to 0.8.1

[boot-splash] Remove displays before unloading plugin

If we don't do this, then the splash display handler will
still be set, causing all sorts of undefined and weird
behavior.

This makes hitting the escape key multiple times act more
reliable.

[main] Use updated terminal attrs on crash

In commit ea394383c54dda771dcf59e7ec11280fa945280b I changed
the terminal settings used by ply-terminal.  This commit puts
those same settings into effect in the crash path.

[main] add more debugging messages

[configure] Add some heuristics to find drm headers

On my system, some headers seem to be stuffed in /usr/include/libdrm
and other headers seem to be stuffed in /usr/include/drm .

I think the ultimate upstream goal is for everything to be in
/usr/include/libdrm but my system seems to be in some transition state.

My pkgconfig files only point me to one of the directories, so add some
heuristics to find the other one.

[client] Allow unlimited unlock attempts

The ask-for-password command has an option
to specify the maximum number of retries to
get the correct password.  The default number
of retries was 3.  This makes sense for /opt,
and to a lesser degree /home, but it doesn't
make sense at all for /, since failing means
the system won't boot.

The new default is "unlimited".  This fixes the
/ case, but has the downside that all integration
code that was depending on the 3 default before
will need to get updated.

[script] Reference this if function is called with current object as this

If calling a function which is connected to the current object, the object
operated on in the function call is the current object. This is rarely used and
somehow survived without causing crashes despite incorrectly not increasing the
refcount.

[drm] Conditionalize fb fallback with 1-head ttm

When using the old style transition code path where
plymouth quits before X starts, we can't use the drm
renderer for single head nouveau and readeon setups
because their kernel memory manager doesn't allow userspace
write access to the console fb and the old style
transition code needs to dump the last frame of the
plymouth animation to the console fb as intermediate step
of the transition process.

The newer plymouth deactivate mechanism doesn't need the
intermediate console fb copy, so we can stick with the
drm renderer all the time.

This commit conditionalizes the "bail to frame-buffer renderer"
check based on whether the build is configured to enable
the old style transition.

[script] Only attach to the keyboard if it has been set

Don't add an input handler to a NULL keyboard.

[script] Connect and disconnect the keyboard on start and stop

Previously we were connecting to the keyboard but not disconnecting from it as
unset_keyboard was not called before the plugin being destroyed. Now we connect
when on splash start and destroy which makes sure we disconnect before being
destroyed.

[drm] Drop drm/ from include directives

They've moved to a different directly with the latest
libdrm, and we should be relying on pkgconfig anyway.

[terminal] Do not discard the input when turning the terminal unbuffered

Drawing to renderers was resetting the terminal to unbuffered, but this was
also discarding any queued input keys. The fix should keep the input buffer
untouched.

[frame-buffer] Initialize backend->head.map_address

It's possible for activate to be called before map_to_device, for
example if a non-graphical splash plugin is in use.  Initialize
map_address to MAP_FAILED so that we won't try to redraw in this
situation.

[main] Watch keyboard input after the splash had a chance to quit

If the splash hasn't picked up any displays it will abort. We wait for the
possible abort before attaching the keyboard. This is a slight hack as now
add_displays_and_keyboard_to_boot_splash no longer adds the keyboards. Also the
splash plugin can access the keyboard before it is officially opened.

For now it just fixes the terminal fd < 0 assertion bug.

[main] watch keyboard input after displays are added

Since the boot splashes handle opening the terminal now,
we need to make sure that we don't try to watch for input on
the terminal until they've opened it.

This should fix some crashes users are reportedly seeing with
the details splash.

A potentially better fix would be to make watch_for_input defer
doing anything on its own using the terminal is opened.  That
would require a trigger and some new api to ply-terminal, I think.

[drm] Add more debug messages

These messages give a clearer view into what's
happening when moving drawing from plymouths
buffer to the kernel's console buffer.

[main] Watch-for-keystroke should also be sensitive to enter

Because enter and general keystrokes are handled separately, enter would not
activate the keystroke trigger. Now, you can watch specifically for an enter
press, and enter is accepted as a general any-key trigger.

[plymouth-set-default-theme] Replace tabs with spaces

[script] Allow alpha color component to be set when generating text images

The Image.Text function now accepts an alpha parameter of the color used. If
not set this will be assumed to be 1 (thus reverse compatible).

[plymouth-set-default-theme] handle old symlink configuration file

[configure] bump to 0.8.1

[client] add update subcommand

This is to replace the old style --update option.

[client] add hide-splash and show-splash subcommands

This is to replace the old style --hide-splash and --show-splash
options.

[client] add new update-root-fs command

This supercedes --newroot and --sysinit into one
command following the newer subcommand method of
using the plymouth client.

[key-file] Add more debug spew

[key-file] Ensure items_matched variable is initialized

The loop condition checks it.  If we bail early because of
comment, we don't want to end up with undefined behavior.

[drm] Add debugging spew

Explain when we fall back to frame buffer renderer on single
head TTM systems.

[frame-buffer] add debug spew

[set-default-theme] Another fix with commented group lines

[key-file] fix debugging message

[key-file] Add support for comments

[main] fixed logic inversion when loading config file

[key-file] Add debugging messages

[main] fix debugging message

[set-default-theme] properly handle commented out [Daemon] section

Before we would assume there was a valid [Daemon] section already
if a comment contained [Daemon] in it.

[main] Fix config file look up properly

Before we were writing the config file splash into the
override_splash_path variable instead of the variable meant for
that config file.

[drm] s/intel/radeon/ in debug statement

[script] Unref the correct element

Typo in previous commit. Should have unreffed the hash and not the function
object.

[script] Strings automatically adopt from the global String template

This means string operations now can be applied on raw strings rather than
having to cast them using the String() class template wrapper.

String("something").CharAt(3)
now becomes simply:
"something".CharAt(3)

[script] Use a pre computed value rather than accessing directly

The name_exp was already set to the thing we wanted, so use that. Less
confusing since other sections use it.

[drm] Don't remove drm fb's that we don't know about

When we access to the kernel console's fb, we don't own
it and shouldn't remove it.

This is like commit 808e129fd144e4065c08314723837b4141e90355,
but for radeon and nouveau instead of intel.

[drm] Add more debug spew

[drm] Compile warning clean ups

[script] Rearrange script_obj_direct_as_hash_element return conditions

Testing for the presence and falling though to return a NULL makes more sense.

[main] Adds a declaration of toggle_between_splash_and_details

Fixes a compile error about a static declaration following an implicit
declaration.

[drm] Add debug spew

[main] Add debug spew

[main] move guts of on_escaped_press to new function

Before we were calling the handler directly when it
wasn't in response to an escape key press event, which
was a little groaty.

[boot-server] Add debugging spew

This will just help down the line if there are problems.

[main] Fix crash when config files don't exist

I wasn't handling NULL return value correctly before.

[script] Silence compiler warning

[daemon] Actually install shipped config files

commit cf766763f26ad0bb8e001a9ecf65b00f4c766e3c was
messed up in more than one way.  The biggest problem
was that It didn't include the Makefile changes needed
to ship the files added to the repository.

This commit fixes that.

[defaults] Drop some debug goo that sneaked in

[branch-merge] Add daemon configuration mechanism

This series of commits adds a few configuration files
to replace the kludgey symlink we use now for theme
specification.

In the future, these config files could potentially
be used for other things as well.

[daemon] Ship default configuration files

This commit adds plymouthd.defaults and plymouthd.conf.

The former is for distributions to override, and the latter
is for administrators to change.

[set-default-theme] Write plymouthd.conf instead of symlink

Now that the daemon looks for the default theme in configuration
files, we should make plymouth-set-default-theme write the
configuration files instead of doing symlinks.

That's what this commit does.

[main] Try getting default theme from config file

Right now we figure out the default theme via a symlink.
This apprach is very simple, but is also a little cumbersome.

It means as the default theme is changed around we have to move
the symlink around.

The symlink is in /usr.  We really shouldn't be mucking with
/usr when changing defaults.

This commit checks the filesystem for two config files:

/usr/share/plymouth/plymouthd.defaults
and
/etc/plymouth/plymouthd.conf

The first one is for distributions to use.  This is how they can
manage which splash to show from release to release.

The second one is for system administrators.  This is how they
can override distribution policy.

We don't actually ship these files yet.  In the mean time,
(and even after for the forseeable future) the old symlink method
will still work.

[configure] drop --with-default-plugin-link

It's an old interface that I don't think anyone is using.

[configure] change bug report link in AC_INIT

It was my email address, but should probably be bugzilla
(or the mailing list)

[configure] run through autoupdate

This just modernizes the file a bit.

[script] Update default script to use the window X and Y offsets every time

Previously the code was assuming the windows were placed at 0,0. This might not
be the case and the window X and Y values should be used when trying to
position items relative to a window.

This change needs to be applied to all other scripts otherwise mutiple screen
setups may have unaligned elements. Updates scripts should be tested using
multi-head test systems or the x11 test renderer.

[script] Correctly initialise the minimum width when getting minimal window

Values were preset to zero which was the smallest width/height.