[3952] Abandon hooks tests on unrecoverable error

Changed some EXPECT_EQ tests to ASSERT_EQ in cases where the length
of a vector was checked and subsequently followed by checks that
accessed elements of the array.  This way, if the array is too short,
the test will cleanly exit rather than possibly segmentation fault.

[3952] Minor comment fixes

[3952] Update hooks-libraries parser tests

These tests were updated while investigating the issue described in
the ticket.

[master] Added ChangeLog entry for #3951.

[master] Merge branch 'trac3951'

[master] Added ChangeLog entry 981 for #3939.

[master] keactrl now uses server PID files

Merge branch 'trac3939'

[master] Merged trac3932 (check remove() return)

[3951] Fixed typo in the D2 logger name.

[3939] Addressed review comments

doc/devel/qa.dox
    Added description of KEA_PIDFILE_DIR

doc/guide/keactrl.xml
    Added discusion of PID files to section 6

src/bin/keactrl/keactrl.in
    start_server()
        Revised wording of already-running log message

[master] Added ChangeLog entry for #3484.

[master] Merge branch 'trac3484'

[3484] Further corrections to spelling and grammar in the documentation

[3484] Miscellanous editing changes during review

Correction of typos/grammatical errors etc.  Also included text in the
contributors' section explaining why test-driven development is used.

[3484] Correct error in example code

[master] Merge branch 'trac3941'

[master] Updated git hash

[master] Finished merge of trac3920 (data_ defined twice)

[3941] Renamed logger objects used by DHCPv4 and DHCPv6 server.

[3939] keactrl now uses PID files to control servers

src/bin/keactrl/keactrl.in
    - get_pid_from_file() - new function which returns a server's PID from its
    pid file, or 0

    - check_running() - modified to use get_pid_from_file(), sets $_pid and $_pid_file
    globals

    - send_signal() - modified to use get_pid_from_file()

    - start_server() - modified log emitted if server is already running to
    contain the PID and pid file

    - stop_server() - new function to stop a given server, IF it is running.
    reload_server() - new function to reload a given server, IF it is running.

    - get_pids() - deleted

    - Replaced use of send_signal with stop_server or reload_server to provide
    better visual feedback to users.
~

[master] Fixed BSD compile error introduced by 3769

Added #include for waitpid() added to testing by 3769.

[3484] Fixed quite a few doxygen warnings.

[3932] Fixed typo

[3484] Small fixes to remaining dox files.

[master] Added ChangeLog entry 978, for #3769.

[3932] Ignored remove return status

[3932] Ignored remove return status

[3932] Ignored remove return status

[master] Servers now create PID files

Merge branch 'trac3769'

[3932] Ignored remove return status

[3932] Ignored remove return status

[3932] Ignored remove return status

[3932] Ignored remove return status

[3932] Ignored remove return status

[3932] Ignored remove return status

[3932] Changed removeFile() to return a status one can check

[3769] Fixed typos in the message files.

[3484] Updated Developer's Guide for libdhcpsrv.

[3769] Addressed review comments

Removed Daemon::init() method, improved server log message
descpriptions, and minor typos.

[3484] Updated sections of the Developer's Guide concerning libdhcp++.

[3484] Updated Developer's Guide for the DHCPv4 server.

[3484] Updated heavily outdated sections about the DHCPv6 component.

[master] ChangeLog updated after #3873 merge.

[master] Merge branch 'trac3873' (FAQ section in the User's Guide)

[3873] Minor word-smithing after review

[master] ChangeLog updated after 3928 merge.

[master] Merge branch 'trac3928' (missing images in dist tarball)

[3484] Updated documentation of the user_chk library.

[3484] Updated hooks maintenance guide.

[3484] Updated Guide for Component Developers.

Mostly nits.

[3484] Updated description of hooks for DHCPv4 and DHCPv6 server.

[3484] Small updates to the Contributor's Guide.

[3484] Moved logo to the top left corner of the developer's guide.

[3484] Replaced BIND10 logo with ISC logo in the developer's guide.

[3769] Updated Kea Admin guide with PID file info

    Added discussion of PID files to start up section for
    each of the servers.

[3769] Removed static from Daemon::config_file_ and proc_name_

Rendered config_file_ and proc_name_ instance members as there is
no reason for them to be static.

[3769] DHCPv6 now uses PID file

Added PID file creation to DHCP6
    src/bin/dhcp6/dhcp6_messages.mes
    -  added log DHCP6_ALREADY_RUNNING

    src/bin/dhcp6/main.cc
    - added logic to create the PID and catch
    exception specific to PID conflict

    src/bin/dhcp6/tests/Makefile.am
    - exports KEA_PIDFILE_DIR

    src/bin/dhcp6/tests/dhcp6_process_tests.sh.in
    - added call to server_pid_file_test

    src/bin/dhcp6/tests/dhcp6_unittests.cc
    - main(int argc, char* argv[])
    sets env var KEA_PIDFILE_DIR

[3769] DHPCv4 now uses PID file, made pid test common

src/lib/testutils/dhcp_test_lib.sh.in
    - server_pid_file_test() - common test for any server
    to verify PID file management

src/bin/d2/tests/d2_process_tests.sh.in
    remmoved duplicate_server_start_test
    now calls server_pid_file_test

Added PID file creation to DHCP4
    src/bin/dhcp4/dhcp4_messages.mes
    -  added log DHCP4_ALREADY_RUNNING

    src/bin/dhcp4/main.cc
    - added logic to create the PID and catch
    exception specific to PID conflict

    src/bin/dhcp4/tests/Makefile.am
    - exports KEA_PIDFILE_DIR

    src/bin/dhcp4/tests/dhcp4_process_tests.sh.in
    - added call to server_pid_file_test

    src/bin/dhcp4/tests/dhcp4_unittests.cc
    - main(int argc, char* argv[])
    sets env var KEA_PIDFILE_DIR

[3769] Added env var,KEA_PIDFILE_DIR; D2 now uses a PIDFile

src/lib/dhcpsrv/daemon.c/h
    Daemon::Daemon() - Constructor will now override the default PID
    directory with the value of env variable KEA_PIDFILE_DIR.  This
    provides a simple means to alter the value for tests.

    Added am_file_author_ flag so Daemon instances will only delete
    a file they have written.

src/lib/testutils/dhcp_test_lib.sh.in
    - verify_server_pid() - new function which verifies that a server has a PID file
    and that it contains the server's PID, and that the process is alive.

src/bin/keactrl/tests/Makefile.am
    - added export of KEA_PIDFILE_DIR to override default PID directory during tests

Added PID file creation to D2
    src/bin/d2/d_controller.cc
    - DControllerBase::launch() - Added block to createPIDFile()

    -DControllerBase::parseArgs() Replaced call to Daemon::init()
    with call to Daemon::setConfigFile()

    src/bin/d2/tests/Makefile.am
    - added export of KEA_PIDFILE_DIR to override default PID directory during tests

    src/bin/d2/tests/d2_process_tests.sh.in
    - dupcliate_server_start_test() - new test which verifies that D2 cannot be
    started twice (with the same configuration file)

    src/bin/d2/tests/d2_unittests.cc
    - main(int argc, char* argv[]) sets environment variable KEA_PIDFILE_DIR
     to override default PID diretory during tests

src/lib/util/pid_file.cc/h
src/lib/util/tests/pid_file_unittest.cc
    Changed PIDFile::check() to return either the PID contained in the PID file
    if the process is alive, or 0, rather than bool.  This permits callers
    to see/log the PID.

[master] Removed spurious io in util::io from doxygen comments (reviewed by jabber)

[3873] Fix typos

[3769] Added support for creating PIDFiles to dhcpsrv::Daemon

src/lib/dhcpsrv/daemon.h/cc
    New methods:
    static void setConfigFile(const std::string& config_file);
    std::string getProcName() const;
    setProcName(const std::string& proc_name);
    std::string getPIDFileDir() const;
    void setPIDFileDir(const std::string& pid_file_dir);
    std::string getPIDFileName() const;
    void setPIDFileName(const std::string& pid_file_name);
    void createPIDFile(int pid = 0);
    std::string makePIDFileName() const;

    New members:
    std::string proc_name_;
    std::string pid_file_dir_;
    isc::util::PIDFilePtr pid_file_;

src/lib/dhcpsrv/tests/daemon_unittest.cc
    New tests:
    TEST_F(DaemonTest, getSetConfigFile)
    TEST_F(DaemonTest, getSetProcName)
    TEST_F(DaemonTest, getSetPIDFileDir)
    TEST_F(DaemonTest, setPIDFileName)
    TEST_F(DaemonTest, makePIDFileName)
    TEST_F(DaemonTest, createPIDFile)
    TEST_F(DaemonTest, createPIDFileOverwrite)
    TEST_F(DaemonTest, PIDFileCleanup)

src/lib/util/pid_file.h
    Added typedef boost::shared_ptr<PIDFile> PIDFilePtr;

[3873] FAQ/troubleshooting section added.

[master] Added ChangeLog entry for #3578.

[master] Merge branch 'trac3578'

[3578] Minor fixes to docs

[3928] Added missing images in src/lib/hooks/images to EXTRA_DIST

[3928] Missing images added to EXTRA_DIST

[master] Added ChangeLog for #3887.

[master] Merge branch 'trac3887'

[master] Added ChangeLog entry for #3740.

[master] Merge branch 'trac3740'

[3740] Changes to the DHCPv4 message description per review comments.

[master] Added ChangeLog entry for #3846.

[master] Merge branch 'trac3846'

[3578] Updated User's Guide per review comments.

Clarifications in the section concerning option values setup.

[3470] Tidy up some typos

[3578] Tidy up some typos in the docs

[master] Fixed distcheck issues with src/bin/admin tests

There were several minor issues related to src/bin/admin tests
that were visible only when distcheck was perfomed with mysql
and/or postgresql enabled.

[3846] Trivial change in the kea-guide.

[3846] Removed the invalid note from the kea-guide.

[3846] Revised DHCPv6 option definitions in the kea-guide.

[3846] Revised DHCPv4 option definitions in the kea guide.

[3920] Applied proposed cleanup of Pkt{,4} buf/len constructor

[3920] Removed reinit of data_

[master] Finished merge of trac3839 (removed unused exception variables)

[3839] copyright

[3839] reindented

[master] Removed spurious lease_(0) introduced by accident in #3919 marge

[master] Added ChangeLog entry for #3913.

[master] Merge branch 'trac3913'

[3913] Addressed review comments.

Moved the function from the anon namespace to the AllocEngine class.

[3913] Allocation engine extends the lifetime of the lease for Request.

[master] Updated git hash

[master] Finished merge of trac3919 (cppcheck small fixes)

[3919] grammar

[3919] copyrights

[master] Merged trac3858a (locat paths in libdhcpsrv_unittests)

[master] Merged trac3922 (Stopwatch not copyable)

[3922] Made Stopwatch not copyable