[1892] Fix the message string in MismatchedPlaceholders exceptions

[1531] Changes after Jeremy's review.

[1528] Nitpick comparison optimization in Linux version of ifacemgr

[1528] Small clean-ups in Linux version of ifacemgr.

[1207] several small cleanups

[1207] rename getInMemoryClientP back to getInMemoryClient

The temporary name with the added P is no longer necessary now the original method has been removed.

[1207] Remove InMemoryClient-specific code from auth_srv

Made the getZoneCount() method from inmemory client a general datasource client
call (which throws NotImplemented by default), as this does seem useful in other
 cases as well.

Now the work starts to pay off; removed the temporary pointer variable (getInMemoryClientP() still returns a pointer but now directly retrieved from the container), and any direct reference to isc::datasrc::InMemoryClient is removed from au
th_srv (but not from tests as of yet)

[1207] remove old-style inmem-ds getter and setter

[master] cleanup: removed an obsolete comment added in development.

It was temporarily introduced at 1fc79b93 for #1028, but it should have been
cleaned up even before commiting the change.
should be trivial and harmless, so I'm directly commiting the cleanup.

[master] Use -c to copy when using install-sh

[1208] use the container to set in-mem client

Instead of the direct shared_ptr.

Changes needed:
Moved loading of zones in config into the memory_datasrc implementation; at this moment the general API has no method for this (and it may not be necessary; datasources should handle their own config anyway).
This does have one drawback; through the dynamic loading mechanism we currently lose exception type information. So for now specific exceptions such as MasterLoadError end up as 'general' DataSourceErrors. Tests have been updated to reflect this.

Also, I made config handler slightly more lenient; it has hardcoded defaults and will now accept config parts without "class" or "zones"; using .spec -derived values is impractical here with the current config API, but in this case it shouldn't *really* matter; as IN is hardcoded in several places anyway atm.

[1803] Document the tests little bit more

[1803] Implement the previousNode start match

In case the find didn't give us an exact node, we need to correct the
search at the beginning. The code is slightly dense, but should be well
commented. Also, the tests are fixed slightly and one more added.

[1803] Special-case handling in previousNode

The empty chain may mean we did not call find previously. But it also
can mean we just run out of nodes or that we did not find the correct
node. We also handle empty tree specially.

[1207] add setInMemoryClient with container instead of shared ptr

[1207] store and use raw inmem client pointer

(temporarily)
In order to add datasource factory, but keep current code working, store both the shared ptr (for reference counting) and the raw pointer it encapsulates. The latter is then used to call query::process().

[1207] add shared_ptr DataSourceClientContainerPtr

[1903] Minor fix to the exception text

It talked about the other method.

[1903] More tests for previousNode

This time, we check it can start iterating if we search for something
that is not in the tree.

[master] Remove files that are put in place by autoreconf

[master] Make a minor whitespace fix to the last but one ChangeLog entry

[1930] Add ChangeLog entry

[1930] Install headers into bind10-specific directory

[1803] Unify some mostly duplicate code

The predecessor and successor are almost the same, so it makes little
sense to have the code twice.

[1803] Implement the previousNode for found nodes

Slight change was done in the test, as the test can't find the "hidden"
nodes using find. Also, we need to clear the node chain manually between
reusing it.

[1892] Add scaffolding check around resource.h

[1803] Tests for part of the previousNode

These test when we start at an exact match. Iterates from the very end
and from somewhere in the middle.

[1803] Interface for previousNode

This is not exactly as in the ticket, but is the same as nextNode.

[1880] Add dontCreateCoreDumps() to EXPECT_ABORT tests

[1880] EXTRA_DIST resource.h

[1880] Style fix

We use anonymous namespace instead of static before a function.

[1880] Move dontCreateCoreDumps() definition to the header due to dependency issues

[1880] Untabify file

[1880] Use a helper function instead of repeating code

[1880] Don't create core files when EXPECT_DEATH is used

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

[master] more ldadd order fixes

other binaries in log/unittests also need to link to our local libs first

Merge #1802

[1892] Return the placeholders to tests too

The test code now reflects the code change.

[1892] Put back the @@Missing placeholder warning when logger checks are off

[1892] Conditionally throw an exception if logger placeholders are not matched

[master] link gtest libs after local libs

so that sys-installed bind10 libraries do not interfere with those in the build tree

[master] Add explicit exception what strings

This should fix the unit tests complaining about wrong error messages.

While debugging, I also found an uninitialized value error that has been present probably since the logger code exists. This has also been fixed.

[1889] use cwd as data_path in cfgmgr if config_file is given

Depending on whether config_file is specified, use either the builtin default or os.getcwd() for the data path (so that you can use relative config file names. If an absolute config file is given, data path is ignored).

Also added an info log message specifying which file cfgmgr ends up using.

[1802] Set the RESULT_NSEC_SIGNED flag in in-memory

When we return a negative answer or wildcard, we return
RESULT_NSEC_SIGNED for NSEC signed zones, similarly to NSEC3-signed
ones.

[1802] Set the NSEC flag on add

Not tested in any way now yet, this is private data structure. It is not
yet used, so the behavior is not changed.

[1802] Tests for NSEC status

The zone should declare itself to be signed if there's an NSEC RR.

[master] Merge branch 'trac1857'

[1856] Use a dummy database_file for lettuce multi_instance test

[1885] Add ChangeLog entry

[1885] Fix lettuce tests, now that b10-stats-httpd isn't enabled by default

[1885] Turn off b10-stats-httpd in the default configuration

[master] update changelog for merge of #1520

[master] Merge branch 'trac1520'

[1857] make other exceptions we define subs of isc::Exception as well

instead of std::exception

[master] update changelog for merge of #1843

[master] Merge branch 'trac1843'

[master] Merge branch 'trac1878'

[1878] use . instead of source

[1520] treat non-existent list item as empty list

instead of treating it as an error

[1878] Remove +x mode from setup_intree_bind10.sh.in

[1843] clean up comments

[master] Update .gitignore

[1843] use re for directive matching

to work with both whitespace and case-insensitivity

[1843] don't revert local config on execute failure

instead, just print some advice

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

[master] Update .gitignore

[master] Put libtool macros in m4macros/

[master] Update .gitignore

[master] Fix endif comment

[master] Changelog update for #1908

[master] [1908] Added v6 socket binding to local-link address

[1540] Removed declaration and initialization of unused variable

... which was left over from a previous change.

[1878] add copyright statement

[1878] add run_lettuce.sh script, and check in terrain

[1878] added run_lettuce.sh script, and check in terrain

[1857] make ModuleSpecError derive from isc::Exception

[master] [1895] editorial: added a missing space in an error message.

not directly related to this branch, but the review comment on this point
for the previous change should actually apply to this.

[master] also wait for auth server to be started before query

This is in the Load and response scenario.

This is to fix lettuce failures see on mac and debian automated lettuce
runs.

This was just a timing race.

Also I didn't check for BIND10_STARTED_CC as done elsewhere
since that never worked. I opened a ticket for that: #1900.

[master] make sure auth server is running before queried

This fixes lettuce failure:

   Scenario: Delegation query for unsigned child zone                                                                      # features/queries.feature:106
     Given I have bind10 running with configuration example.org.inmem.config                                               # features/terrain/bind10_control.py:107
+    And wait for bind10 stderr message AUTH_SERVER_STARTED                                                                # features/terrain/steps.py:34
     A dnssec query for www.sub.example.org type AAAA should have rcode NOERROR                                            # features/terrain/querying.py:204
     Traceback (most recent call last):
       File "/usr/pkg/lib/python2.6/site-packages/lettuce/core.py", line 117, in __call__
         ret = self.function(self.step, *args, **kw)
       File "/home/reed/work/isc/bind10/git/bind10-CLEAN/tests/lettuce/features/terrain/querying.py", line 240, in query
         "Expected: " + rcode + ", got " + query_result.rcode
     AssertionError: Expected: NOERROR, got NO_ANSWER

This was discussed on jabber. (Maybe the cmdctl check is not needed,
but shouldn't hurt either.)

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

[1843] import fix

Fix makefile

The run_unittests should be linked only if it is compiled. Confirmed on
jabber this was a mistake.

[1843] add test for change to find_spec_part

and a bit of comment cleanup

[1843] move command_sets/execute initialization

to command_sets, so that all setup is done in one file and not 3

[1843] revert script config changes if script fails

[master] Merge branch 'trac1704'

[1704] Corrected a typo in the bind10-guide.xml file after review

Also regenerated the bind10-guide.html and bind10-guide.txt files.

Merge #1793

[1793] Fix a comment

The old version is a result of copy-paste and is wrong.

[1793] Eliminate several state variables

The values are deduced from something else, no need to keep them. This
eliminates the ugly initialization with wrong values.

[1793] Fix variable name

We have camelCase in many places, but variable names are not one of
those.

[master] changelog for #1891

[master] Merge branch 'trac1891'

[1843] lettuce test for builtin init_authoritative_server

[1843] half-related cleanup; return copies of spec parts

so that if they have nested default values, which may be set to 'local', and modified later, the original isn't changed. (technically it should be enough to do the copy when setting, but it's safer to copy them entirely for now)

[1891] enabled some DatabaseClientTest for add/del NSEC3s for SQLite3, too.

They were originally planned to be enabled with the completion of this branch,
but I forgot to enable them before review.  As expected, they just passed
for SQLite3, too.

[1891] added bindXXX methods to StatementProcessor as sqlite_bind_xxx wrapper.

As suggested in review.

This will make the caller code simpler.  It's also safer in that the caller
doesn't have to manipulate the statement separetely (in the previous way
StatementProcessor and the caller could use different statements).

Also made naming more consistent: in the very initial version the class was
called "Executer" and then renamed to Processor.  Some variables of this class
were still named "exec", etc, but it's more consistent if they are named
like "proc".

[1843] add a builtin command set for execute

moved exception catching around a bit, so the set does not get completed if one of its steps fails

[1528] Modifications to some of the comments