Implement mock functionality for some system calls in bp_sys.

Stub out system functions for testing (WIP).

This is a more thorough and should be an overall better approach than
stubbing/mocking the base_proc_service class itself.

This commit introduces a baseproc-sys.h header which holds aliases for
the system functions that are used (kill, fcntl, etc). When running
tests the functions will be mocked (currently they are rigged to abort
when called).

Move 'run_child_proc' function into a separate source file.

Move base_process_service timer expiry handling out of friend class.

One less friend class is a good thing. The timer now just calls a single
public 'timer_expiry' function. The latter function can also be used in
tests.

Add some process-service tests.

Use general dinit.h header in baseproc-service.cc.

Makefile: build dependency lists (*.d) during regular build.

No need to build the dependency files as a separate step.

Also, make "make check" no longer build main objects.

Stub out Dasynq functionality for tests.

Later some of this may be turned into proper mock functionality.

Add more missing includes for tests.

Fix makefile typo preventing build of 'shutdown'.

tests: infrastructure for implementing mock headers.

We need to duplicate the header directory entirely in order to be able
to replace header 'xyz.h' (so that 'include "xyz.h"' in other, real,
headers resolves to the mock).

Fix some includes (for testing/general resilience).

Move includes into a separate directory.

Minor cleanup: remove unnecessary aliases.

Pull some commonly occurring declarations into a new header file.

Adds a new header file, dinit.h, which includes declarations for some
types and functions exported from dinit.cc.

Tests: use stubs for various functions.

proc-service.cc: add intro comment.

Refactoring: split out exec_succeeded function.

The status pipe watcher (exec_status_pipe_watcher) had some
service-record-type specific code. Split it out as a virtual function.

Move separate_args function from header to source file.

This avoids potential ODR violations. The function isn't worth making
inline, it's large enough and not on a critical path so just compile it
once.

Fix build of tests.

Split service.cc into multiple files.

service.cc becomes service.cc + proc-service.cc + baseproc-service.cc.
The header service.h becomes service.h + proc-service.h.

This refactoring should make testing easier.

Fix issue with start timeouts not correctly stopping dependents.

Big rename/namespace cleanup.

Minor refactoring/code documentation.

Properly handle scripted service start interrupt.

If we issue an interrupt (SIGINT) to a starting script, but the script
completes normally, we should run the stop script before marking the
service as stopped.

Clean up warning in process_restart_timer.

Initialise the service pointer via the constructor to avoid warning.

Rename service_type (type) to service_type_t.

This is consistent with a number of other service-related types, and as
a bonus makes building with GCC 4.8 possible (it was previously confused
by a "service_type service_type = ..." declaration).

Linux examples: use reasonable start-timeout for filesystem services.

Allow specifying service start timeout in service description.

Default is 60 seconds.

Correctly handle start timeout of 0 (i.e. no timeout).

Implement start timeout for services, default of 60 seconds.

If a service doesn't start within the timeout it is sent SIGINT, and
then SIGKILL after the stop timeout (if there is one).

Sample service defs: enable smooth recovery for ttys.

Also change dependency on ttys from depends-on to depends-ms.

Update version to 0.07.

Install: allow specifying sbin and man directories.

Put man pages in the correct directory (man8 not man1).

Add "start-interruptible" service option.

This option allows a service startup to be interrupted if the service
becomes inactive while starting, by sending it the SIGINT signal. This
is useful for file system checks, which can otherwise delay
reboot/shutdown.

Allow decimal comma separator for times expressed as decimals.

Some locales use a comma to separate a decimal whole from its fractional
part, so allow either a decimal stop or a decimal comma.

Fix minor style inconsistencies.

Add "tests" generated binary to .gitignore

Use _Pragma to turn off invalid-offsetof warnings at appropriate places.

This means that we no longer need to build with -Wno-invalid-offsetof.

Add some missing includes, necessary on some platforms.

Allow for interrupting process startup (by sending SIGINT).

Refactoring: Move fields from service_record to base_process_service.

service_record: encapsulation: make some members private, add accessors.

Build and run tests with sanitizers enabled.

Merge pull request #1 from mskarbek/tests_fix

Fixed tests after start_ps_process and all_deps_stopped rename.

Fixed tests

Fixed tests after start_ps_process and all_deps_stopped rename

Renamed 'start_ps_process' function to 'bring_up'.

Also some minor reorganising, improve comments etc.

Rename 'all_deps_stopped' function to 'bring_down'.

The former name doesn't imply the action to be taken.

Make do_start/do_stop protected instead of public.

Handle fork+exec failure in a separate virtual method.

This cleans up the code and removes a lot of conditional handling for
the different service types.

Typo fix.

Fix potential issue stopping process services.

Process services undergoing smooth recovery count as "STARTED" but have
a pid of -1, or a valid pid but with a process launch underway (waiting
for exec status). If all dependencies stopped at the wrong time, the
behaviour was to treat the service as stopped, but the smooth recovery
was still kicking on and could cause an issue if it "succeeded" after
the service was supposedly terminated.

To resolve this, don't send a kill signal to a process during smooth
recovery (i.e. if waiting_for_execstat is true) and send it once
recovery has completed instead.

Various minor cleanups.

Build: use EXTRA_LIBS when building "shutdown".

bgproc_service: remove redundant initialisation from subclass.

Update comparison on similar systems.

Update Dasynq to 1.0.3.

Update Dasynq to 1.0.2.

Upgrade bundled Dasynq to 1.0.1.

Incorporate upstream changes from Dasynq.

This updates bundled Dasynq to 1.0 (plus minor changes since).

Add some tests for milestone dependencies.

Define and use a macro to run a named test.

Add "depends-ms" dependency option for milestone dependencies.

Correct use of SOFT vs WAITS_FOR dependency type in various places.

Also fix implementation of WAITS_FOR. SOFT presently behaves the same as
WAITS_FOR (but cannot presently be specified as a dependency type in a
service description).

Simplify dependency checking logic.

Add service set destructor.

Destructor deletes all services in the service set. This fixes a memory
leak currently present in the tests.

Re-work dependency specification for service_record constructors.

Dependencies of different types are now given together in a single list,
wth each dependency tagged by its type, rather than as two lists (one
for regular and one for soft dependencies). This opens up the
possibility of supporting additional dependency types.

Improve a test (and make it pass).

When a service is unpinned from pinned start, it needs to check
force_stop and stop if set.

Fix test 6, add a new test (test 7).

Keep all dependencies (soft and regular) in a single list.

We had separate lists for regular dependencies, dependents, and soft
dependencies and dependents. Combining the two dependency types
simplifies code in several places and allows for adding new dependency
types much more easily.

ss_read: silence signed vs unsigned comparison warning.

tests: add another.

service: fix recently introduced double-start bug.

Tests: add another test.

Fix "make check" on OS X / BSD.

tests: correct get_state call (after rename from getState).

Dasynq: merge changes from upstream.

TODO list: a couple of items done.

More camelCase to under_score renaming.

Consolidate linux-specific code.

Various renaming of camelCase to underscored_name for consistency.

(Linux) set dinit as child subreaper.

This allows catching/watching double-forking processes even if we
are not PID 1. Double-fork orphans will be reparented to the
subreaper (i.e. Dinit) rather than PID 1 (init).

Use getpgid to determine process group of service process.

Using pid (i.e. 'kill(-pid, signo)') doesn't work if the process has
double-forked but not set a new process group for the child, and
anyway it's not clear if POSIX actually guarantees that process group
IDs will match process IDs.

smooth recovery: wait for dependencies to start before relaunch.

If pinned started, dependencies may be stopped; the smooth recovery
process should wait for dependencies to start before trying to re-launch
the service process.

bgprocess: eliminate doing_recovery field.

We can use the existing 'restarting' instead after some minor changes.

TODO: update

Fix spacing / remove hard tabs.

service descriptions: Implement the stop-timeout setting.

service: implement stop timeout / hard kill.

If a service fails to stop in its alloted time, it will be killed with
SIGKILL. The timeout is currently hardcoded to 10 seconds.

service: check error from kill when checking ability to track child.

We check whether the process exists using kill. We were treating any
error return as meaning the process ID was invalid, but in fact we may
simply not have permission to signal the process (if we are not being
run as root and the service process is suid).

service: add safety check before unlinking old socket path.

service: improve robustness of read_pid_file().

Cleans up some TODOs.

service: use recorded state to check whether we can track child process

We have a flag which records whether we are tracking the child process.
A BGPROCESS service can't always track the child, so it stops once the
dependencies have stopped without waiting for the process. Previously we
did a manual check rather than using the existing flag to see whether
the process could be tracked before immediately stopping.

Makefile: fix warnings on initial build due to missing *.d files.

These are the auto-generated dependency information files, which do not
exist on initial build (or after clean).

process service: more fixes to child watch reservation.

Various renaming (from CamelCase to underscore_separated).

For consistency with most of the rest of the code base.

Improve code documentation.

Fix child watch reservation.

Fix minor typos