Improve unit test infrastructure - allow control of write()

In the unit test infrastructure, allow behaviour of the write() call
(i.e. bp_sys::write()) to be overridden with a custom handler.

dinitcheck: fix nullptr deref if service can't be loaded

Add a simple logging test

Use bp_sys aliases to write from log

This allows for future unit testing of the log infrastructure.

Mark main log closed on write failure

Record the main log as being closed (external_log_open false) if there
is a write failure while trying to write the log buffer. This allows the
log to be re-opened by restarting the logging service.

Also we avoid waiting for the log buffer to flush if it has failed.

dinitcheck: check for missing service command

Add int. test for failing to load service

Flush main log (as well) before exit

The console log is flushed before exit (with a timeout of 5 seconds),
but the primary log was not flushed until now.

Fix issue when service fails to load

Introduced during earlier refactoring.

Fix error messages from proc_service launch stages

BUILD: explain the USE_UTMPX option a little better

query_name: always return with NAK if service handle invalid

Existing code looks like a copy-pasto: returns ACK (not NAK?) and falls
through to a potential null-pointer dereference if the service handle
can't be found.

Add SHUTDOWN_PREFIX build variable, controls name of shutdown executable

Useful for installing Dinit alongside eg Sys V init.

Avoid building constant strings on the stack

In a few places cts::literal was used locally within a function,
resulting in the (constexpr) result string object being allocated on the
stack. Move these to namespace scope.

Add dinitcheck(8) to dinit.8 "see also" section

Updated bundled Dasynq to 1.1.7

This fixes a slow memory leak.

Fix --help option in dinitcheck

Update COMPARISON; give more details of Dinit up-front

Avoid copying string arguments to log functions (always pass by ref)

Bump version.

dinitctl: make unload/reload respect --quiet

dinitcheck: output overall result after checks complete

Push some TODO items back to 0.9+

We already have two major features in 0.8: dinitcheck, and dinitctl
reload; that's enough for a new version.

dinitctl man: note that can't add non-running dep to started service

Clean up clang warnings

Update default build configs

Refactoring: load new / reload existing service in single function

Loading a new service and reloading a service share a lot of common
code, so refactor them into a single function.

Fix IGR makefile (spaces to tab)

Update TODO, minor updates to COMPARISON/DESIGN

Fix reload of stopped service

Add 2nd reload test

Set target state to STOPPED when stop is issued

Add integration test for reload

Implement additional checks for reload

The settings/flags inittab-id, inittab-line, shares-console,
runs-on-console, and pid-file cannot be changed for a running service.

Improve some dinitctl error messages

Remove unneeded release

Implement reload command for dinitctl

Implement control protocol for reloading services

Initial implementation of service reloading

dinitcheck: return EXIT_FAILURE if problem found

Integration test for dinitcheck dependency cycles detection

dinitcheck: check for and report dependency cycles

Fix inconsistencies/typos in help text, man pages

dinitcheck: support specifying services to check on command line

Initial manpage for dinitcheck

Update TODO

Install dinitcheck on install, clean binary on clean

Remove accidental marker comment

Add integration test for basic dinitcheck functionality

dinitcheck: use previously factored-out parameter processing

dinitcheck now shares parameter processing with dinit, providing
consistent error messages and parsing.

Factor out service parameter processing

Factor service parameter processing into a template method, to enable
sharing (later) between load_service in dinit and dinitcheck.

Service description loading: improve error messages

Include parameter name and improve consistency in error messages when
parsing service descriptions.

dinitcheck: fix checking of dependencies

Use common option processing for dinit/dinitcheck service dir

Factor out service directory option processing

Service directory options need to be processed/handled by dinit and by
dinitcheck, so factor out the processing into a separate module.

Fix typo in comment

dinitcheck: process all dependencies

Don't enable reporting exceptions on failbit (only to then disable).

Beginnings of "dinitcheck" utility

This will be a utility to check service configuration for errors/lint.

Convert system_error to service_load_exc in load_service.

More re-wording of "dinit as init" Linux documentation

Restore -s option to dinitctl

This is hardly needed since it is the default if run as root now, but is
nice to have for backwards compatibility / overriding -u option.

Update gitignore

Remove integration test output

Merge pull request #22 from travankor/makefile

Fix missing dependencies on mconfig.h in Makefile.

Fix makefile.

Merge pull request #23 from travankor/doc-fixes

Fix typo in dinitctl man page. Document restart for help option.

Fix typo in dinitctl man page. Document restart for help option.

Merge pull request #21 from fpoussin/contrib

Adding fpoussin to contributors

Adding myself to contributors

Merge pull request #20 from fpoussin/root

Adding root check on default control socket

Closes #18

Updated documentation for the new user flag

Replaced system flag with user flag

Merge pull request #19 from fpoussin/modules

Kernel modules script

Adding boot link for modules

Adding kernel modules script

Up version to 0.7.0.

Some documentation updates.

Add integration test for service "restart" function.

Update man page for revised restart/wake semantics.

Change semantics of "wake" command.

"Wake" now re-attaches dependencies (from the dependents of the named
service). You can no longer wake a service if it has no active
dependents.

Avoid signed-unsigned comparison warning.

dinitctl: update restart handling.

As per changes in dinit daemon/control protocol.

Re-work restart function.

Restart is now always "gentle", i.e. cannot be forced and isn't possible
when dependents (with hard dependency) exist.

Properly issue STARTCANCELLED message if restart is cancelled

Re-work state propogation/transition logic.

This mostly keeps existing behaviour, hopefully making it more
consistent and predictable in other cases, and simplifies the code.

Improve/refactor static_membuf.

Refactor some packet-building for readability and safety

Document dinitctl "restart" command.

Update TODO.

Up version, update NEWS.

stop: if not required, treat stop as full manual stop.

This is needed to properly handle shutdown when a service is alive but
not marked active.

Implement dinitctl restart, addresses #14

Don't add console watcher if not PID 1.

If not PID 1, we don't need to ask for console input. Adding a watcher
fails if we don't have a (suitable) stdin, eg FreeBSD shell if dinit is
run in background.

Update manpage/README for new "dinitctl stop" semantics.

"dinitctl stop" will now issue an error if dependent services would be
forced to stop. The new --force option can be used to force a service
and dependents to stop.

Add integration test for gentle stop/force stop.

Fix a bunch of issues identified by Coverity scan

dinitctl: print names of dependents which are inhibiting stop

dinitctl stop: gentle by default, implement --force.

A "gentle" stop won't stop if dependents will be affected, i.e. it warns
and aborts. The `--force' option (or `-f') can be used to force the
service to stop, which will also cause the dependents to stop.

Also sneak in a bug fix where certain options were not recognised.

Add control protocol query-name test