Merge branch 'trac3119'

[3119] Minor cosmetic cleanups as per review

[3087] Added state handler unit tests for D2::NameAddTransaction

Created unit tests for all of the states implemented by NameAddTransaction
in b10-dhcp-ddns.  The class lacks only proper construction of DNS update
requests.

[2940] Fixed handling of the NULL client identifier in the Memfile backend

[2945] Update Doxygen reference to isc::ConfigData::getFullConfig() (contd.)

[2945] Update Doxygen reference to isc::ConfigData::getFullConfig()

[3107] Move CXX_VERSION initialization closer to where it is used

[master] Add ChangeLog for #2300

[3087] Initial implementation of d2::NameAddTransaction

This interrim checkin includes the initial implementation of
NameAddTransaction to b10-dhcp-ddns.  It provides the preliminary
state model needed to add forward and/or reverse DNS entries for
and FQDN.  It does not yet construct the actual DNS update requests.

[3087] Use IOServicePtr consistently in DHCP-DDNS

Some classes were using references to isc::asiolink::IOService, others
where using d2::IOServicePtr.  The latter is now used throughout for
consistency as well as support for future, possible, multi-threaded
implementation.

[master] Add lettuce test for mixed-case query

[3119] Added a comment regarding autoconf macros used.

As per review added a comment about the use of $ac_header_preproc.

Merge branch 'trac2300'

Conflicts:
src/bin/xfrin/b10-xfrin.xml
src/bin/xfrin/xfrin.spec
tests/lettuce/features/xfrin_notify_handling.feature

[master] Update emptiness checks to be more efficient (cppcheck)

Reviewed by Thomas Markwalder on Jabber.

Merge branch 'trac3172'

Conflicts:
src/bin/dhcp6/dhcp6_srv.h

[master] fix xsltproc --stringparam html.stylesheet usage

This is to set the location of the css stylesheet.
Previously it pointed to a path in the source directory.
By default, this was "./" so was okay as a location via HTTP.
But when built using a different source directory, then
the full source directory ended up as a reference on the webpage
so then style's didn't work.

I briefly discussed this on jabber yesterday but didn't get an okay.
It is a minor fix, so committing per my discretion.

[master] spelling fix in docs

(minor fix, committed without review)

[2945] Add missing doxygen comments

[3172] Fix build failure on BSD systems

The comment at the end of adding to AM_CXXFLAGS was making the total
line too long, and the automake tool was splitting the line. This meant
that the macro assignment had an escaped comment, which the BSD make
interprets differently to GNU make (the latter swallows the next line as
part of the comment, whereas the former doesn't).

Merge #2932

Receiving of notifications in C++.

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

Conflicts:
ChangeLog

[master] Updated ChangeLog

[433] Merge branch 'trac433'

[530] Do not redirect b10-msgq STDOUT when running in verbose mode

When bind10 is run in verbose mode, as is indicated to the user by
certain error messages, do not redirect b10-msgq's STDOUT to /dev/null
so that the user can see the actual cause for startup failure.

No ChangeLog entry required.

[530] Display socket file used in error message

When there is an error connecting to the message queue socket file
because it wasn't cleaned up properly, and we know the name of the
socket file, display its name as part of the error message.

No ChangeLog entry required.

[master] Added Changelog entry for #3183.

Also additional trivial fixes applied: fixed the Changelog entry number
for #3211 and git commit id for #3199.

[master] Merge branch 'trac3183'

[3207] Clean and doxygen commentary for user_check hook library.

Adding doxygen and cleaned up code for initial review submission.

[3183] Minor: added a todo comment in perfdhcp.

[3210] Fixed trivial issues in the CfgMgr.

This commit includes trivial changes to the doxygen documentation and
make one variable const.

[3073] Merge branch 'trac3073'

[3207] Added support for default users and delimited user id input

Vendor option default values are now taken from default users if the
client is not registered.  Registry input file user ids can now contain
colons as delimiters allowing cut-and-paste from outcome file.

[3207] Added support for TFTP_SERVER option to user_chech hook library

User check hook library now supports adding TFTP server option to DHCP
response packet options from values in user registry.

[master] ChangeLog updated

[3211] Changes after review:

 - ChangeLog clarified
 - Comments in option_definition.h corrected

[3211] Incoming IAPREFIX is now parsed properly.

[3207] Added bootfile option output to user check hook library

Added ability to add the boot file option from the user registry to
the outbound DHCP response to user check hook library.

[3207] Link fix in user_chk

[master] ChangeLog update after 3194 merge.

[3207] Replaced selected subnet with lease address in user check output

User check hook library's outcome output is now generated in pkt_send
callout allowing the output of lease address (or prefix) in place
of selected subnet.

[master] distcheck fix after 3194 merge.

[3183] Typo correction and one todo added.

[3183] Addressed review comments.

[master] Merge branch 'trac3194_1'

[master] Fixed libdhcp++ unit test to use correct buffer length for compare

Okayed on jabber.

[master] ChangeLog updated.

Merge branch 'trac3201'

[3201] Constants moved to dhcp4.h

[3201] BIND10 Guide updated.

[3194] Unnecessary s-file removed.

[3194] Cleanup in the OptionVendor class.

Added consts where applicable and extended comments.

[433] Verify the existing daemon behaves correctly after multiple invocations

As per review, update the tests to ensure that the existing message
queue daemon still works even after we have had a failed attempt to
start a second copy of the daemon.

[3194] Do not duplicate options in responses from the server.

[3194] Append requested vendor options only if the lease is assigned.

Also, fix getting requested option codes from ORO.

[433] Use a local variable rather than one larger in scope

As per review, don't use self._session for the check to see if there is
an existing message queue running. Rather, use a localally scoped
variable that will get cleaned up when the function exits.

[3207] Added callouts for packet receive and send to user check hooks lib

Initial checkin creates callout functions for packet receive and send
hooks for IPv4 and IPv6.  This sets thes stage for outputting user specific
options in outbound packet.

[3194] Merge branch 'master' into trac3194_1

[3194] Minor fixes in libdhcp++ - add consts where applicable.

[3210] Docs updated.

[master] Fixed make install failure in user_chk unit tests

Yet another make issue introduced with trac3186.  Unit test data files
are now looked for in the appropriate directory and copied as part of dist
check.

[3210] client-id echo in DHCPv4 server is now configurable

[3210] CfgMgr now can store a flag for echoing v4 client-id

[3201] Test that suboptions are parsed correctly for fixed-size options.

[3194] Unit-test for vendor option implemented.

[3201] Add support for DHCP options that hold only sub options.

[master] Add ChangeLog for #3094

[master] Delete trailing whitespace in ChangeLog

Merge branch 'trac3074'

[master] Fix typo

[master] Delete trailing space

[master] Fixed unit test build issue in user check library

trac3186 added the user_chk hook library which had a couple of
uses of EXPECT_EQ on stl containers. Gtest 1.5 does not like this.

[master] Added ChangeLog entry 697 for trac3186

[master] corrected distcheck issues from trac3186

After merging, distcheck revealed a few minor issues with logging
files and unit test data files. These were corrected.

[3194] Changes after review:

 - config parsers updated
 - new unit-tests for vendor options written
 - libdhcp++ cleaned up
 - fixes in option_vendor.cc
 - comments added and cleaned up

[master] Merge branch 'trac3186' DHCP hook library user_chk

Adds the hooks subdirectory to bind10/dir for storing hooks shared
libraries, and the first such library for DHCP, user_chk.

[3186] Addressed review comments.

Mostly minor cosmetics. Added logic to select callouts to
simply return if no subnets are configured.

[master] Merge branch 'trac3191' (next-server field in DHCPv4)

Conflicts:
ChangeLog

[3191] Typo fixed.

[master] Merge branch 'trac3195' (DHCPv6 unicast sockets)

Conflicts:
ChangeLog

[3194] Check if option definition for Vendor option has correct format.

[master] Added Changelog entry for #3200.

[master] Merge branch 'trac3200'

[2246] Interface detection code for BSD and Solaris

 - Contributed by David Carlier
 - Clean-up by tomek

[3194] Use OptionDefinition::optionFactory to create DHCPv6 vendor options.

[3194] Use OptionDefinition::optionFactory to parse V4 Vendor option.

[3195] Changes after review:

 - Guide updated
 - Iface::addUnicast() now refeses duplicates
 - addUnicast and clearUnicasts are now commented
 - new unit-tests
 - argument for addActiveIface is const reference again
 -

[3200] In one more AssertionFailure pass the error message as parameter.

[3200] In two additional AssertionFailures, pass the message as parameter.

[3200] Pass error message as a parameter of AssertionFailure.

[433] Improve test and make status variable local in scope

[3191] Changes after review

 - default value is no longer 0.0.0.0, so global value is no longer
   overwritten with subnet-specific values
 - added 2 new unit-tests to check if the value is really set in
   server responses
 - guide updated to explain that
 - fix in Dhcp4ParserTest.nextServerNegative

[3200] Changes after code review.

[3194] Implemented basic vendor options support.

[3073] Rremove incorrect checks for empty() that caused test failures

[3073] Rremove incorrect checks for empty() that caused test failures

[master] Added ChangeLog entry for #3199.

[master] Merge branch 'trac3199'

[3199] Added option 66, 67 to bind10-guide and change the type of option 2.

[3186] Added test_users_err.txt test file for user_check lib

Added unit test data file, overlooked with prior commit. Also
added missiong include to user_file.cc and took extraneous debug
cout.

[3200] Test that NAK lacks requested options.