Merge #1944

This fixes the logging placeholders in messages to really match what is logged.

[2009] Fix narrowing warnings with GCC 4.7

[1890] Clarify message shown when command to clear the config file is used

[1457] fix a few final todo's

most important, a catchall during processing of the update section

[1457] whitespace fix

[1457] refactor: don't return tuple from __get_update_zone()

[1457] refactor: make new members private

[1457] refactor: initialize ZoneFinder once per session

[1457] refactor: make zname, zclass and datasrclient member variables

instead of arguments that need to be passed to nearly every method

[1457] refactor ddns session unit tests

made a local easy function to create rrsets, which should make most tests much easier to read (less clutter around rrset creation and rdata addition).

Also changed the tests added in #1455 to use this

[1457] finish ddns update unit tests for code coverage

[1457] docstrings

[1457] tests for rrset class convert function

[1457] log message descriptions

[1457] more CNAME update tests and fixes

[1457] rename helpers, add cname update tests

[1457] initial addition of log messages for update section

[1457] added test, and check for dupe rrs

[1457] initial update handling

Still needs a lot of cleanup, logging and more tests, but the basics are there

[master] (re)add the LIBRARY_PATH_PLACEHOLDER trick for dhcp6 tests

now its python test relies on isc python modules we need this workaround
for MacOS X.  This fix solves recent buildbot failures on macmini.
okayed on jabber

[master] Merge branch 'trac1456'

[1503] -p portnumber option implemented for b10-dhcp6

 -p portnumber command-line option implemeneted for b10-dhcp6
 Corresponding tests implemented as well.

[master] Unnecessary header removed from iface_mgr.cc

  - Compilation fix for FreeBSD

[master] added missing copyright statement

This fix is trivial, so directly committing without review.

[1944] Don't use shared method for logging

At least for some logging. This eliminates having more real parameters
than placeholders for the used messages. Other occurrences of the
function are preserved, as the corresponding messages have full set of
placeholders.

[master] Removing unused_id variable (compilation fix for perfdhcp)

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

[master] ChangeLog numbers update, after 1281 merge.

[master] Clean fix for unused variables warning.

[1456] addressed review comments

- refactored duplicate code
- updated docstrings

Merge branch 'trac1281'

[1704] Add some tests for InterprocessSyncNull class too

[master] Workaround for unused-variable issues on Solaris.

[master] Use clockid to avoid "unused variable" warning.

[1956] Created PerfPkt6 class

New class handles DHCPv6 messages in perfdhcp tool

Merge branch 'trac1954'

[1704] Use 'logger' as the task name for all interprocess sync objects inside log/

[1954] Minor changes

Remove a duplicated line in commandOptions::reset() and corrected
header comments in initClientsNum().

[1704] Check that logger acquires and releases locks correctly when logging messages

[1704] Add note about exception thrown

Merge #1948

This is the guide update to mention the listen_on workaround for multi-homed
hosts.

[1704] Set B10_LOCKFILE_DIR_FROM_BUILD for various tests

[1704] Use a null InterprocessSync object for some tests

This makes some tests pass which uses logger_example, which is
a standalone binary.

[1704] Update documentation

[1704] Set and use B10_LOCKFILE_DIR_FROM_BUILD in tests

 * Also set this env variable in isc::log::initLogger()

[1704] Remove log_test_messages.{cc,h} from the tree

They are auto-generated during build now.

[1704] Add targets to create log_test_messages.{h,cc} from .mes

[1704] Throw if NULL is passed to logger.setInterprocessSync()

[1704] Use @abs_top_builddir@ directly in the .py.in file instead of using an env variable

[1704] Assert that the lockfiles exist

[1704] Assert that the lockfile_test directory exists, and is removed

[1948] Make minor editorial changes

[1704] Move test directory creation to setUp() / tearDown()

[1704] style matters: don't omit {}, func name convention, typo.

also use unnamed namespace instead of file-scope static.  technically, this
use of static was deprecated in C++.  also use the unnamed namespace for
the entire TESTs, which is our common (though not explicitly documented)
convention.

[1503] DHCPv4 Python tests now don't block when nothing is printed on stderr

[1503] ChangeLog updated.

[1954] Correcting error in -D<value%>

[1456] remove double empty line

[1954] Updated with suggestions from second round of review

[master] Merge remote-tracking branch 'origin/trac1455'

[1455] Merge branch 'trac1455' of ssh://git.bind10.isc.org/var/bind10/git/bind10 into trac1455

[1455] add new test data files to EXTRA_DIST

[1456] fix a few comments and error messages

[1456] bit more explanation about single_update_mode

[1944] Deactivate logger on exception

We want to disable the logger if there was an exception while putting
the parameters in. While we can't detect all possible ones, we can
detect some and reduce some false positives of the strict log checker
and not output mangled messages.

[1944] Make sure we print the message ID on abort

When we enable the strict log checking and the excess placeholders check
triggers, we need to print the message to easily identify the message ID
with the problem.

[1944] Add a missing placeholder for number of labels

The DATASRC_DATABASE_FINDNSEC3_COVER logging message was missing a
placeholder. Added.

[1455] Added notes about possible optimizations

As per review comment http://bind10.isc.org/ticket/1455#comment:15

[1456] Add 'single-update-mode' to xfrin/diff.py

This mode is useful for DDNS-type updates; it allows for additions and deletions to be done in any order (keeping track of them separately).
It does have some extra restrictions; SOA records must be the first deleted and the first added type, and may not occur later anymore (causes ValueError).

For now, I left this in python/isc/xfrin, even though it is not purely xfrin anymore. Should we move it to datasrc? or util?

[master] also link with libutil_io

This fixes problem noticed with
g++ (Debian 4.6.3-2) 4.6.3
GNU ld (GNU Binutils for Debian) 2.22

The linker no longer indirectly links. The linker no loner
searches for dependencies.

This one line addition patch shared on jabber yesterday; builder had problem
again today.

[1704] Test remove_lock_files()

[1458] added documentation for ACL related methods of ZoneConfig.

[1704] Use B10_FROM_BUILD instead of B10_FROM_SOURCE (contd.)

[1704] Use B10_FROM_BUILD instead of B10_FROM_SOURCE

[1704] Don't set B10_FROM_SOURCE for logger unittest

[1704] Add documentation for InterprocessSync and InterprocessSyncFile

[1704] Add documentation for setInterprocessSync()

[1704] In case NULL is passed as the sync object, do nothing and return

[1704] Move code to different places (no functional changes)

[1704] Add a comment about use of select() in the test

This commit also:

 * Moves the parent's code to read the lock state into a helper
   function.

 * Describes the 5 second timeout as an arbitrary value.

[1704] Initialize sync_ in the initialization list

[1704] Update comment about using locking in the logger

[1704] Remove obsolete include

[1704] Make InterprocessSyncFile's destructor virtual

[1458] sort the log messages.

[1458] supported update ACL with TSIG.

on thinking about this in more detail, I deprecated UpdateSession's req_data.
the new assumption is that the caller should complete TSIG validation,
and for ACL check UpdateSession just uses the stored TSIG record in the
given message.

[1458] handle the case of ACL action being DROP.

[1458] first step for __check_update_acl: check ACL, and handle reject case.

[1704] style matters: long line, () for return, const, { position.

also change 'const string' to 'const string&' in some places

[1948] Suggest listing all addresses in listen_on

The guide is modified to suggest to list all the addresses explicitly
for a multi-homed server, since the wildcard addresses don't work well
yet.

[1455] make shallow copy of to-be-modified list

[1704] Use top_builddir during tests as top_srcdir may not be writable

[1704] Unit test the logger for the lock

[1704] Open lock files lazily

[1704] Add a select() to the parent reader in case the child messes up

[1704] Replace TEST_F with TEST

[1704] Minimize code

[1704] fd_ cannot be -1 here

[1704] Use a helper for lock/unlock/tryLock as they use the same pattern

[1704] Remove separate class hierarchy for InterprocessSyncLocker