[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.

[script] Get window value without an index returns area covered by all windows

Calls to Window.GetWidth/Height/X/Y without a window index now return the
values of the area covered by all windows. This is only the case if all the
windows are aligned (either by their centers, or to a corner).

This allows the theme designer to place an object knowing it will be seen on
all screens.

[script] Return NULL when requesting width of nonexistent window

Minor bug, previously would return the index used rather than a NULL. Would
only cause problems when using a width request as a test of the presence of a
window.

[script] Move windows to not start at negative coordinates

When multiple screens are found, the system will now arrange them so they are
all centered, and the top left corner of the largest screen is at 0,0.
No changes to any scripts are needed.

[main] Deactivate terminal on quit if necessary

If we're done with the VT plymouth was running on,
and plymouth wasn't running on the initial VT, we
should jump back to the initial VT and try to
clean up plymouth's VT.

[terminal] Add new ply_terminal_deactivate function

This will jump off the VT associated with the terminal
and deallocate it.

[ply-text-display] don't set mode on every write

Resetting the mode to text on every write means that if you're
using a text plugin and X starts, X's VT keeps getting reset back to
KD_TEXT since those plugins don't stop writing on deactivate (they
have no renderer).

There's no reason to set this mode here anyway; all paths to using
those plugins already do this.

[terminal] keep track of terminal raw/cooked state

Instead of setting the terminal to unbuffered (raw) mode on every
write, keep track of whether it's unbuffered or not at the points
we open and close the terminal.

Deactivate already takes care to set back into buffered mode;
otherwise we can end up resetting the terminal mode under X causing
Enter to send X SIGQUIT.

[main] deactivate renderer before hiding

If we don't deactivate the renderer before hiding the splash, the
drm renderer may scan out the buffer contents to the fbcon buffer;
since we only hide the splash when dumping details or when
--retain-splash is *not* given to quit, this is exactly the
opposite of what we want.

The effect of not doing this is partial splash contents behind the
details in cases of error, or when using quit.  This doesn't affect
plymouth quit --retain-splash.

[daemon] add "has active vt?" request

One problem with the current deactivate/quit transition into X is that
the display manager will, if Plymouth was running, re-use the currently
active VT.

That only works if Plymouth was actually displaying a splash screen on
that VT.  If --show-splash hasn't been called yet because we booted too
fast, we'll be on the wrong VT.

Add a request to ask whether the Plymouth VT is active; I've done it
this way so the answer defaults to "yes" for Fedora who use VT1.

The pseudo-code for transition is thus:

  if plymouth is running (ping):
    plymouth deactivate
    if plymouth has active vt:
      start X on current VT with -nr
      if X starts ok:
        plymouth quit --retain-splash
      else if X fails:
        plymouth quit
    else if plymouth doesn't have active vt:
      plymouth quit
      start X as normal
  else if plymouth isn't running:
    start X as normal

[client] add "has active vt?" request

This asks the daemon whether it has an active VT, used for a smooth
transition into X.

I've implemented this to look like ping.