Minor TODO updates.

service start_dep(): only bump acquire count if not already holding.

dinitctl: enable: correct argument order for symlink

Target, then link name, apparently.

dinitctl: correct logic for checking missing arguments.

Enable/disable commands can set 'to_service' without setting 'service';
check for this.

Allow '.' in setting names (for "waits-for.d").

Minor build fixes for Linux/Glibc.

dinitctl man: add enable and disable command descriptions.

dinitctl: implement "disable" command.

Opposite of "enable"; removes a dependency and unlinks the entry in
waits-for.d of the target service.

dinitctl: refactoring.

dinitctl: implement "enable" command.

This is for persistently enabling a service. The enabled service is
added as a dependency of its target (default "boot") and an entry is
added to the target's waits-for.d directory (which must be specified
exactly once in its service description).

tests: add test for control protocol ENABLESERVICE command.

control: add function to enable service.

This adds a dependency, and starts the dependent service if not already
started (and if the dependent is started/starting).

Factor out service description loading to a separate header.

This allows for reading service descriptions directly from dinitctl,
which is useful for planned "enable/disable service" functionality.

Reduce some source line lengths.

Move service type id constants into service-constants.h

dinitctl: reduce verbosity of help text, trim some lines.

dinitctl: add utility method to extract string from cpbuffer.

control: send current working dir in load mechanism reply packet.

The load paths are relative to the current working directory, so the
client needs to know what it is to resolve the paths properly.

cpbuffer extract(): use integer arithmetic instead of char pointer.

The method is designed to extract data to an arbitrary type, and while
char * should probably be safe to do that with, conversion to char * is
not specified in C++ as it is in C, and compilers are becoming more
aggressive.

control: server side support for querying load paths.

Support for querying the service loader mechanism (currently there is
only one) and its details (the paths from which it loads service
descriptions).

dirload_service_set; correctly record service dir allocation.

The service directory can be allocated statically or dynamically (in
which case it is "owned" by the service set). Correctly track it for the
single-directory constructor.

dinit: remove unused variable.

dinitctl: unload: issue a specialised message if service not loaded.

dinitctl: fix incorrect exit status if start/stop fails to load service.

dinitctl: give an error message if unrecognized option/command given.

An unrecognized option was just exiting silently, and an unrecognized
command would print the help text. Now, print an appropriate error in
each case.

Avoid multiple cout statements when printing help.

Print help as one big string rather than by one-per-line cout with endl.

Updated bundled Dasynq to 1.1.5.

Remove some default copy constructors and their unintended uses.

Build docs: document fuzz testing for control protocol.

Tests: use consistent output spacing and flush before running each test.

Avoid undefined char* arithmetic.

Use uintptr_t instead of char* for "unusual" pointer arithmetic.

Tests: add another test.

Add a test that starting a service which has a waits-for dependency on
another service, currently stopping, causes that service to restart
after stopping.

Adjust dependency auto-break logic.

If a service stops, and has a soft dependency from another service,
don't break the dependency if the service is waiting_on this one - in
that case we want to restart.

Minor header/signature adjustments.

Add const to some method arguments.

Documentation: overcommit must be enabled for address sanitizer (Linux).

Tests may run with address sanitizer enabled, so overcommit must be
enabled or address sanitizer disabled.

Minor documentation updates.

build: add target to run cppcheck.

Fix a number of minor issues found by static analysis.

service options: add waits-for.d to automatically add dependencies.

The waits-for.d option allows specifying a directory, and the names of
files within are added as waits-for dependencies (excludes names with a
leading dot).

tests: add control test (add/rm dependency).

control: ensure shutdown type is valid.

tests: make unload test more comprehensive.

Specifically, make sure that the service that we request to unload is
not loaded afterwards.

tests: add control protocol test (unload).

tests: add control protocol test (start/stop service).

tests: add another control protocol test (load service).

Add fuzz testing for control protocol based on LLVM libFuzzer.

tests: add some protocol tests.

tests: fix build deps for cptests.

testing: small fixes.

Add missing includes.

Compilation on Mac OS fails due to these.

build: re-arrange to build all tests then run all.

This is better than building the main tests and running them, and then
building the control protocol tests and running them, since you see all
test results together.

tests: add c.p. test for list services command.

Add contribution guide, including conduct statement.

Some updates to the design document.

Minor README fix.

Add initial control protocol test.

This includes a lot of framework necessary for writing additional tests.

Minor cleanup of header.

dinitctl: add rm-dep subcommand to remove dependencies.

control: add command for removing dependencies.

build: remove test executables on clean.

build: generate mconfig.h in include directory.

dinitctl: add-dep subcommand, adds dependency between services.

service: correctly roll-back state on exception (bad_alloc).

Since a service adds its dependencies to other services' dependents
lists, it needs to roll back those changes if failing due to a bad_alloc
(or other exception).

Implement adding dependencies via control protocol.

Correct a comment.

Add DESIGN overview.

dinitctl: fix spacing in output.

mconfig: improve consistency of comments.

dinitctl: fix includes for FreeBSD.

Update TODO.

Bump version to 0.3.0.

Tests: add test for loading a non-existent service.

Add "signal-process-only" option to avoid signalling process group.

docs: minor update to build instructions.

dinitctl: show if service was skipped or failed.

Also update documentation accordingly.

Correctly report failure to find a service description.

Remove -fno-rtti from MacOS builds.

It seems to break exception handling. (Didn't I remove this earlier?...)

Replace README with markdown version.

Add more detail to sample "dinitctl list" output.

Since pid and console state are now displayed, add them to the sample
output.

Define constants for supported control protocol versions.

(Rather than hard-coding the values at point-of-use.)

dinitctl: show process id and console state in service list.

Move stopped_reason_t to service_constants.h header.

service: add various accessors.

Avoid rebuilding tests each time they're run.

Tests: add a new test.

Remember the reason why a service stopped.

This will later allow for reporting the reason to dinitctl etc.

Avoid another unnecessary issue-stop-on-release.

Avoid issuing do_stop twice when a process service process exits.

Probably it's harmless, but we may as well avoid it.

Don't try to stop an already stopped service.

It might or might not actually cause a problem, but it will at least
perform some unnecessary work, so don't do it.

Refactor: rename onstart_flags_t to service_flags_t.

Refactor: move "start-interruptible" flag into onstart_flags

Tests: add a test for interrupted start of skippable scripted service.

Add test for scripted service start skip.

Implement "skippable" option for scripted services.

This causes a service start that is skipped via interruption (eg ^C) to
be considered as started for dependency purposes.

TODO: add planned features for next release.

Up version to 0.2.0.

Fix bug causing scripted service shutdown scripts to be run twice.

All tests now passing.

Add another process service test, currently failing.

Don't keep char * from array_string temporary.

The string might not be resolved as a compile-time constant, and the
temporary object may be destroyed before the string within it is used.

array_string: declare 'arr' member as const.