[master] Merge branch 'master' of ssh://git.bind10.isc.org/var/bind10/git/bind10

[master] Merge branch 'trac2211'
with fixing Conflicts to:
src/bin/auth/auth_srv.cc
src/bin/auth/auth_srv.h
src/bin/auth/datasrc_clients_mgr.h
src/bin/auth/main.cc
src/bin/auth/tests/auth_srv_unittest.cc
src/bin/auth/tests/command_unittest.cc

[master] Use std::copy to write IP address to buffer.

This fix resolves build issues on Fedora 17. Okayed on jabber.

[master] Remove narrowing conversion from int to uint8_t.

This fix eliminates build failure on Fedora17 and Debian7. Okayed on jabber.

[master] Merge branch 'trac2210'

[2211] adjusted log severity from ERROR to FATAL before aborting.

[2211] removed an extra block from sendCommand() with clarification comments

[2211] let the thread abort if it sees an unexpected exception, rather rethrow.

update the test case and log message accordingly.

[2211] s/swapDataSrcClientLists/setDataSrcClientLists/ and changed the behavior

the users of this method only need the set behavior, and the previous code
actually didn't swap them as expected anyway.

[2211] style fixes to test JSON configs

[2209] Hack the ZoneTableSegment into MemoryClient

This branch needs the MemoryClient to have its Zonetablesegment. But it
doesn't have one now and it is not the task of this branch to put it in
there. So I added it inside in somewhat ad-hoc manner.

This is hairy and adds bunch of temporary methods. If there's a better
approach, let's use it. But taking this for now to be cleaned up in some
follow-up ticket.

[2209] Unrelated fix: don't c-string + int

It doesn't produce the expected result (which was obviously expected
in the code). Changed to lexical_cast, which should be enough.

[2316] Replace constructor with default param values with constructor
without parameters.

Having constructor with default params values caused build issues on
Mac/clang which complained about not providing constructor's params and ignores
default ones.

[2210] update catch/error logs in datasrc reconfigure

[master] Merge branch 'trac2304'

[2236] Remove obsolete comment

[2236] Don't use PTHREAD_MUTEX_ERRORCHECK in non-debug builds

Also disable tests in non-debug builds that depend on
PTHREAD_MUTEX_ERRORCHECK.

[2236] Remove config.h include from sync.h

... and move it to sync.cc. This is so that sync.h can be dist-ed.

Also add comments to some methods that may not be available in
non-debug builds.

[2236] Add comments to configure.ac about --enable-debug and performance

[2236] Access Mutex::locked() only when ENABLE_DEBUG is defined

[2208] Pass RRClass to ZoneTableSegment::create() factory method

[2361] Update comment about local_plugins directory

Revert "Merge branch 'trac2198_3'"

This reverts commit e6ba20b0907ab7e861e608a71a35f71ab228e79f, reversing
changes made to c9ba8373e271fb3c966fb59813c21ca8c575354d.

[2342] Corrections to comments in configure.ac

[2342] Incorporate Trac2142 changes

[2211] added notes about ownership on the returned value of findClientList().

[2342] Enable MySQL support with flag in configure.ac

[2211] wording/grammer fixes to documentation and comments.

[2211] use swap instead of splice. for this purpose it's enough and simpler.

[2209] Tests for the getCachedZoneWriter

They are the reload() tests reused for this purpose. They fail, since
the method is temporary and contains almost no code.

[2342] Added deleteLease6 functionality

[2209] Interface of the getCachedZoneWriter()

Unlike what the ticket suggests, it takes concrete origin parameter, not
some yet unknown config value, for two reasons:
 * The format of config is not known.
 * Leaking some details about what memory segment or whatever is being
   used for the cache through the interface looks like a bad idea. Even
   leaking the existence of the cache is enough.

Revert "[2209] Interface of the getCachedZoneWriter()"

This reverts commit 812392465ae1ffa4a583186fd48f4c7aff1640dd.

This should have gone to a branch, not to master. Reverting it until it
gets reviewed properly.

[2209] Interface of the getCachedZoneWriter()

Unlike what the ticket suggests, it takes concrete origin parameter, not
some yet unknown config value, for two reasons:
 * The format of config is not known.
 * Leaking some details about what memory segment or whatever is being
   used for the cache through the interface looks like a bad idea. Even
   leaking the existence of the cache is enough.

[2209] Make the reload tests typed

This'll allow us to reuse them for testing the ZoneWriter approach for
the same purpose as reload(). The ZoneWriter one is not there yet, so
this is an equivalent change functionality-wise.

[2210] rename DataSrcClientListsPtr to ClientListMapPtr

[2210] Address review comments

as discussed in the ticket comments

[2304] Fixed build issues on Mac and BSD systems.

[2342] INSERT and GET tests now working

... after moving creation of MYSQL_BIND structures to a separate class.

[2342] Move creation of MYSQL_BIND array for lease addition to separate class

[2304] Removed enum initializers for DataType values.

[2208] Untabify

[2208] Use consistent naming for ZoneTableSegment variables

[2208] Don't use .get() syntax in some places in managed ptrs

[master] Execute run_dbutil.sh using shell

This is necessary because run_dbutil.sh sometimes loses its execute
(+x) mode bit when regenerated, and tests fail due to it.

Merge branch 'trac2198_3'

[2198] Move get methods into unnamed namespace

[2198] Fix library paths to add lib/util/threads/ too (Mac OS test failure)

[2198] Avoid static destruction fiasco with InterprocessSyncFile and logger

[2198] Check for EDEADLK too when using Mutex::tryLock()

[2198] Link -lpthread in libb10-threads target

[2198] Don't return anything from thread functions

[2198] Update comment

[2198] Test the non-blocking variant of Mutex::Locker

[2198] Check pthread_mutex_trylock()'s return value

[2198] Revert include to local

[2198] Stop using direct lock methods on Mutex inside InterprocessSyncFile

[2198] Fix include filename

[2198] Make direct methods on Mutex private

[2198] Add a block argument to Mutex::Locker's constructor

[2198] Add AlreadyLocked exception to Mutex::Locker

[2198] Remove dependency on libb10-util in src/lib/util/unittests/

[2198] First use a mutex for mutual exclusion among threads, then a lock file

[2198] Add Mutex::tryLock() and make lock methods public

tryLock() is necessary inside InterprocessSyncFile. It also doesn't
make sense afterwards to make a dynamically constructed Mutex::Locker
that lives after the basic block is exited. So lock() and unlock()
have also been made public.

I've added comments to discourage the use of these methods directly
and use the Mutex::Locker where applicable.

[2198] Test that locks work from threads

This test fails now because there's no mutual exclusion among threads
in the implementation.

[2198] Build threads directory first because . depends on it

[2342] Now have addLease and getLease6 methods apparently working

Add a virtual destructor

To fix compilation.

Merge #2207

"define and implement (datasrc::memory::)ZoneUpdater class"

It is now ZoneWriter.

No changelog.

Conflicts:
src/lib/datasrc/memory/Makefile.am

[2207] Note about compilation problem

[2211] (unrelated fix) make sure the first time config is installed

it's a topic of #2363, but the change is small and is better to be fixed
here because it would cause conflict.

[2211] added note about exception cases in remote config handler.

[2211] cleanup: removed now-unused stuff: auth_srv mutex, client lists, etc.

[2211] use data source client mgr for query_bench

[2211] use DataSrcClientsMgr in the remote config handler for data_sources.

[2211] test duplicate mgr holder acquisition instead using auth locker

the latter will soon be deprecated.

[2211] use DataSrcClientsMgr for query processing

[2211] make DataSrcClientsMgr and DataSrcClientsBuilder non copyable.

they are not expected to be copied, and I actually had a bug due to an
accidental copy.  Making it non copyable will help avoid such bugs.

[2211] updated auth LoadZoneCommand so it uses the new DataSrcClientsMgr.

[2211] introduce DataSrcClientsMgr::Holder.

auth_srv test's clientList tests are now converted to clients manager's test.
("holder" tests).

[2211] create the client lists map in the DataSrcClientsMgr ctor.

this will help corner case considerations of it being NULL

[2211] added trivial getter for data source clients manager.

[2211] moved DataSrcClientsMgr object from main to auth_srv

this way we can avoid having a situtation that it's NULL.

[2211] added reconfigure() for DataSrcClientsMgr

[2207] style fix: folded a long line.

[2207] Extend the strong-exception guarantee test

We try to put some data there and see it survived.

[2210] editorial cleanup: removed a redundant blank line.

[2324] AllocEngine is now part of b10-dhcpsrv lib.

[2210] style fix: adjusted brace position

[2210] (unrelated) cleanup: removed now-unnecessary include.

[2207] Exceptions & asserts

Changed an exception for unexpected problem and removed a comment about
assert.

[2207] Documentation update

[2207] Use state variable

Instead of several boolean variables.

[2324] LeaseMgr tests improved.

[2210] style fix: use const& for catch

[trac2361] typo fix in comment

noticed this while editing file for previous commit

[trac2361] fix issue where the bind10 could not run in source tree

when the installation was not done yet.

I added extra local_plugins path and custom datasrc.spec plugin for that.

[2361] Make minor whitespace updates

[2210] assorted cleanups

A few more checks in test
cleaned up added headers
reordered log message file

[2210] move DataSrcClientListsPtr typedef

from AuthSrv:: (in auth_srv.h) to isc:datasrc:: (in client_list.h), the previous location wasn't really relevant now that it is used elsewhere, and this seemed the most logical location