GitLab

Conflicts:
	doc/devel/mainpage.dox
	src/lib/dhcp/Makefile.am
	src/lib/dhcp/lease_mgr.cc
	src/lib/dhcp/lease_mgr.h
	src/lib/dhcp/tests/Makefile.am
	src/lib/dhcp/tests/lease_mgr_unittest.cc

A number of other files were modified to resolve incompatibilities
in the way lease managers were created, and to promote the dummy
"memfile" lease manager to a fully-fledged lease manager with
unit tests.

Having duid.{h, cc} in libdhcp++ causes dependency between two libraries.
Due to this dependency multi-threaded build fails because libs are created
by different threads simultaneously and race condition occurs. Libraries
should be moved to separate folders and have separate makefiles. This
complies with BIND10 structure and will make this issue goe away.

For some reason, on Solaris this causes types "uintXX_t" to become
undefined.  The "using" statement was removed and an explicit
namespace prepended to declarations of shared_ptr and dynamic_pointer_cast.

The allocation engine is now pointed to by a shared pointer.  Previously
it was pointed toi by a raw pointer and not deleted in the destructor.
Also, use an automatic variable to hold the DHCP6 server in the
unit tests.

From http://bind10.isc.org/ticket/2468#comment:2 to help troubleshoot later.

Okayed via jabber.

a) Corrected some method headers
b) Added additional diagnostic output should the test fail to
   open the database.

... to ensure that it is always called when a method terminates.

and suggest checking output from msgq using -v.

previously it could result in exception (by the attempt of killing sockcreator
as a chuser'ed process) during initial setup failure, leading to unclean
shutdown.  the revised code prevents it from happening, and also makes the
code concise by unifying the common logic.

so, it now just prints some hints to stderr and exits.

it even suggests restarting as a root.

it now includes textual representation of errno, and, if it's EACCES
("permission denied") the messsage suggesting restarting as a super user.

this is necessary for Solaris. okayed on jabber.

DNS_SERVERS option is now sent only if requested by clients
 (as it should be)

Conflicts:
	src/bin/dhcp6/dhcp6_messages.mes
	src/bin/dhcp6/dhcp6_srv.cc
	src/bin/dhcp6/dhcp6_srv.h
	src/bin/dhcp6/tests/dhcp6_srv_unittest.cc
	src/lib/dhcp/addr_utilities.cc
	src/lib/dhcp/tests/cfgmgr_unittest.cc

Files automerged successfully:
    src/lib/dhcp/alloc_engine.cc
    src/lib/dhcp/subnet.h
    src/lib/dhcp/tests/alloc_engine_unittest.cc

 commit.

do this only when stdin is a terminal; otherwise some lettuce tests will fail,
and it would possibly break some other automatic processing scripts.

based on review discussions.  it makes sense as we cannot assume its address
is never needed in non test applications.