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

[master] forgot CLEANFILES entry in merge of #963

this should fix the distcheck failure reported in http://git.bind10.isc.org/~tester/builder/BIND10-distcheck/20120327113000-FreeBSD8-i386/logs/distcheck.out

[1715] Implement the missing unset method

The underlying implementation is already there, it's just somebody
forgot to wrap it in this another layer of indirection class.

[1715] Tests for the unset method

The method is clearly missing, that's why the command fails. Creating
tests for when the method exists.

[1715] Test the unset command

It's shocking how much is not tested here. No surprise the missing
command was not discovered. Still, not adding tests for the others,
they'd probably fail and it would make the ticket a lot bigger.

[master] Update changelog for merge of #963

[master] Merge branch 'trac963'

[master] update changelog for merge of #1443

[1443] Merge branch 'master' into merge1443

two options have been added to boss's __init__(), conflict resolved.
Conflicts:
src/bin/bind10/bind10_src.py.in
src/bin/bind10/tests/bind10_test.py.in

[1491] copy default named-set data to local when editing

if a child member is set, and the named-set data is default, it would 'lose' the other default values if it wasn't copied to local. For succesful operations this wouldn't be a problem (as this only happens 'locally' in bindctl), but it can cause problems when updates fail due to mistyped data

bug #1819: Add manpage note that the --no-kill option was introduced for testing

bug #1819: Add some comments to testcases

bug #1819: Check that BoB doesn't kill during shutdown when asked not to

bug #1819: Add testcase for --no-kill and -i options handling

bug #1819: Document --no-kill option in bind10 manpage

bug #1819: Update help text for no-kill option

bug #1819: Rename --no-kill-i option to --no-kill

bug #1819: Add ChangeLog entry

bug #1819: Pass nokill as an argument to BoB constructor

bug #1819: Change conjunction used in help message

bug #1819: Add boss nokill flag (-i/--no-kill-i)

[1772] use IPv6 address for xfrin test.

See the previous commit.  This solves test failure on some MacOS environment.

[1772] used ::1 instead of 127.0.0.1 2nd bind10 instance and the example test.

127.0.0.1:47807 is used by a MacOS application and the conflict makes the
test fail in environments that uses the app.  Using an IPv6 address is not
a cleanest solution, but unless we make it fully configurable we need to
handle this type of thing by some workaround anyway.  Also, I believe
it's good to use IPv6 transport for some tests in any event.

[1772] supported IPv6 server address in query test in the form of [v6_addr].

note that this change should be backward compatible.

[master] fix mistake in xml tag

filename argument is required

No review. Typo is obvious. Not sure why xsltproc didn't complain though.

Commit regenerated version too.

[master] changelog for #1829

[master] Merge branch 'trac1829'

[1836] also added unittest that would fail the fixed in this branch.

To make this test workable, I needed to change the returned flag
for the NXRRSET case in MockZoneFinder::find() from setting NSEC_SIGNED
unconditionally to setting any SIGNED flags.  this doesn't affect other
tests.

[1836] added a lettuce test case for delegation from unsigned zone with DO bit.

this would identify the bug fixed in this branch.

[1836] throw exception from addDS only when the find result != NXRRSET.

Otherwise, queries with DO bit can incorrectly trigger an exception if
the zone is unsigned and queyr result is delegation.

[963] addressed review comments

documentation changes
different exit status on uncaught exceptions
removed unused log message
updated 'corrupt' log message
improved 'quiet' test case

[1443] docstring update for backup file

[1443] cleanup and docstring for new test

[1443] append .X if backup config file exists

where X is 1, and increases until a nonexistant file is found

[master] update changelog for merge of #1172

[master] Merge branch 'trac1172'

[master] Update .gitignore

[master] fix indentation consistency in ChangeLog.

[1829] make sure Session.send() transmits all data even after partial write.

[master] noteed RBTree::nodeFission should preserve the current node's name.

This will help at the phase of revisiting the design/implementation.

[master] Merge branch 'trac1775'

[master] changelog for #1688

[master] Merge branch 'trac1688'

[963] forgot EXTRA_DIST line

[963] add --quiet mode to b10-dbutil

[963] do --verbose through log (re)init

[963] use logging framework for b10-dbutil

Except for when the user is prompted for Yes or No, all output now goes through
the logging framework

[963] fix type in b10-dbutil manpage source

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

[963] improve exit status

0 on success (or upgrade aborted by user)
1 on check and need update
2 on check and version of db too high
3 on command-line error
4 on db read error
5 on upgrade error

[963] fix run_dbutil.sh script to not change cwd

with this change, you can pass run_dbutil.sh a relative path name for the db file

[963] fix b10-dbutil.8 creation rule add it to git

[1579] more editorial and style (guideline) fixes

[1579] more editorial fixes

[1579] constified is_nsec3

[1579] suggested change for an awkward comment line:

- s/NSEC3PARAMETER/NSEC3PARAM/
- "definitely" is too strong in this context
- add a space after a comma
- "zonefile" is awkward (there's no "zonefile" in this context)
- "NSEC3 method" is an awkward term
- "this zone signature" is also awkward

[1579] editorial: removed spaces at EOL; folded long lines, () after return.

[1775] ensure wildcard expanded RRsets are considered of same kind if they are.

in the previous version, if one is returned from find() and the other is
from getAdditional() they are not recognized as of the same kind, because
the underlying pointers are different.

[1775] cleanup: make sure wildcard and glue don't coexist.

findNode() should have ensured that, so assert() is used for this.

[1775] handled corner cases with wildcard expansion: empty or multiple nodes.

The previous version crashes if the wildcard node is empty.  Also, the
previous version didn't work well if there were multiple wildcard matches
because the auxiliary tree changes as we add more nodes and stored
pointer can become invalid.  The fix to this issue is a bit complicated;
I needed to make the process two-stage.

[1688] (unrelated cleanup) make RESERVE_RRSETS a private member of Query class.

This will prevent it from causing definition bloat even if the linkder
doesn't combine the constant def.  It would also make more sense to make
it inaccessible from others anyway, because it's basically an
implementation-internal constatnt.

[1688] use referene, rather than making a local copy, for a const vector.

[1688] removed a redundant newline

[1688] use sstream, not iostreaam, for ostringstream.  also reordered headers.

[1688] removed an (now) unused header file.  also reordered headers.

[1443] Addressed review comments

- if self.clear_config is None was wrong, should just be if self.clear_config
- 'back up' should have been 'create backup of'
- default value for unused parameter rename_config_file in b10-cfgmgr_test.py.in MyConfigManager should have been False, not None
- reordered message file

[1172] fix the cmd parser list index problem

make the initial if>2 part of the while
also add a space when glueing extraneous arguments together

[master] Change log file maxsize setting in the BIND 10 guide example

bug #1825: Don't use log4cplus macros which are going to be obsolete soon

LOG4CPLUS_OPEN_MODE_TYPE and LOG4CPLUS_FSTREAM_NAMESPACE are going to be
removed in log4cplus 1.1.0 according to the bug report.

[1680] update changlog entry for this ticket.It is 2 weeks ago
that this ticket merged into master.

[1579] adjust codes after discussion with jinmei. append more unit
tests for negative case in find function

[master] Merge branch 'trac1293'

[master] Add another .gitignore rule

bug #1728: Add -avoid-version in some more places

bug #1728: Set LD_LIBRARY_PATH on OpenBSD as well

master: Fix AC_LANG_SOURCE warnings from automake

bug #1728: Add -avoid-version to LDFLAGS for Python modules (contd.)

bug #1728: Add -avoid-version to LDFLAGS for Python modules

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

[master] document Xfrout notify command.

And remove the now renamed/removed internal only zone_new_data_ready
command.

Regenerate nroff file too (which includes

No new changelog entry. This goes with changelog entry 402.

Somewhat discussed on jabber. Didn't get consensus on one sentence,
but that can always be clarified later.

[1775] use a template to share the same code  for const/non cost of findNode.

we can then avoid const_cast.

[1775] added some more notes about the property of the aux wildcard tree.

[1775] added an assertion check for NULL pointer.

this shouldn't happen based on the RBTree API contract, but some versions
of cppcheck reportedly complain about this.

[1688] re-enabled lettuce tests that failed due to missing dup suppression.

note: there's an error in the test description itself.  In "7.2.2 other",
the result should be NOERROR, not NXDOMAIN, because it should match
*.w.example (BIND 9 indeed returns NOERROR, for that matter).  I fixed
this, then all tests passed.

[1688] removed redundant duplicate check for NSECs.

[1688] use BOOST_FOREACH instead of dedicated for loops.

this could be slightly more efficient because it avoids redundant calls
to vector::end().  In my experiments I actually didn't see much difference,
but it doesn't harm anyway.

[1688] suggested reorg: merge response building to the inserter class.

that way we don't have to expose mutable member variables of `Query`.
now the responsibility of the "inserter" is enlarged, also renamed it
to represent the new job more appropriately.

[1688] additional test case of duplicate in the same section.

the implementation already correctly handles this, so no need to update it.

[1688] suggested cleanup: use vector::insert instead of copy().

also avoid referring to an invalid range of iterator with operator+; use
end() instead (although the original code was probably safe at least in
practice).

[1688] some more trivial cleanups: long line, comment wording, simplification.

[1688] editorial fixes/cleanups: folded long line, naming convention, constify.

also removed "addRRset" utility from rbnode_rrset_unittest, which seems to
be a leftover from an intermediate version (and isn't used anyway)

[master] fill in TBD for the git hash of #1727

Also removed the garbage '?' mark.  It appears I accidentally committed/pushed
the proposed entry to master...so this is just for cleanup.

[master] Merge branch 'trac1727'

bug #1639: Update ChangeLog

[1443] add command-line arguments

also extend log message a bit

[1443] add method to move config file

[master] change DATASRC_QUERY_NO_ZONE from INFO to DEBUG

using debug level of 40 for now.

This is for ticket #1102.

Also clarify the log description.

No changelog entry.

Discussed some on jabber.

[1293] update ZONEMGR_UNKNOWN_ZONE_FAIL description

[1172] better printing of command help

- don't reorder output based on mandatoriness
- copy 'item_description' field from spec file as well

[1172] check for SyntaxError exception

and reraise as DataTypeError (which is handled), so bindctl does not crash