[1584] trivial editorial fixes to meet the coding guidelines.

[trac1687][master] make --disable-man work

This was broken due to misuse of setting.
(--disable-man caused it to be enabled still.)

This may be changed in trac1687 branch to have different default behavior.

[trac1687]Merge branch 'master' into trac1687

[master] make --disable-man work

This was broken due to misuse of setting.
(--disable-man caused it to be enabled still.)

This may be changed in trac1687 branch to have different default behavior.

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

[master] add missing space in item descriptions

okayed via jabber

[1593] a couple of style fixes

[1613] constify

[trac1687] remove generated roff man pages

Later steps will make sure these are generated for "make dist"
to be included in tarball.

[trac1687] remove generated guide and message documentation files

Later steps will make sure these are generated for "make dist"
to be included in tarball.

[master] update changelog

[master] Merge branch 'trac1649'

[1613] fix comment

[1643] Split the add_remote_config gut to a function

[1593] Addressed comments from review

[1613] data mangle wasn't necessary, reverted

library: Fix pkglibdir of backends

autoreconf errored out with:
src/lib/datasrc/Makefile.am:34: `pkglibexecdir' is not a legitimate directory for `LTLIBRARIES'

and consecutive errors such as:
src/lib/datasrc/Makefile.am:43: variable `memory_ds_la_SOURCES' is defined but no program or
src/lib/datasrc/Makefile.am:43: library has `memory_ds_la' as canonical name (possible typo)
src/lib/datasrc/Makefile.am:36: variable `sqlite3_ds_la_SOURCES' is defined but no program or
src/lib/datasrc/Makefile.am:36: library has `sqlite3_ds_la' as canonical name (possible typo)
src/lib/datasrc/Makefile.am:45: variable `memory_ds_la_LIBADD' is defined but no program or
src/lib/datasrc/Makefile.am:45: library has `memory_ds_la' as canonical name (possible typo)
src/lib/datasrc/Makefile.am:39: variable `sqlite3_ds_la_LIBADD' is defined but no program or
src/lib/datasrc/Makefile.am:39: library has `sqlite3_ds_la' as canonical name (possible typo)
src/lib/datasrc/Makefile.am:44: variable `memory_ds_la_LDFLAGS' is defined but no program or
src/lib/datasrc/Makefile.am:44: library has `memory_ds_la' as canonical name (possible typo)
src/lib/datasrc/Makefile.am:37: variable `sqlite3_ds_la_LDFLAGS' is defined but no program or
src/lib/datasrc/Makefile.am:37: library has `sqlite3_ds_la' as canonical name (possible typo)

According to automake "bug#7647: Diagnose invalid primary/directory combinations"
as in http://www.mail-archive.com/bug-automake@gnu.org/msg01927.html

we nowadays have to use pkglibdir for LTLIBRARIES, not pkglibexecdir.

+2010-12-17  Stefano Lattarini  <stefano.lattar...@gmail.com>
+
+ Warnings about primary/prefix mismatch fixed and extended.
+ * automake.in (%standard_prefix): Add `doc' and `locale'.
+ Rename `pkgdatadir' to `pkgdata'.  Similarly for`pkglibdir',
+ `pkgincludedir' and `pkglibexecdir'.
+ (handle_programs): List `pkglibexec', not `pkglib', among the
+ prefixes valid for the `PROGRAMS' primary.
[]
+for primary in $primaries; do
+  prefixes_ok=''
+  case $primary in
+    LIBRARIES|LTLIBRARIES)
+      prefixes_ok='lib pkglib'
+      ;;
+    PROGRAMS)
+      prefixes_ok='bin sbin libexec pkglibexec'
+      ;;

Signed-off-by: Bernhard Reutner-Fischer <rep.dot.nop@gmail.com>

configure: conditionally add AM_SILENT_RULES

Use AM_SILENT_RULES if your automake is new enough (1.11 or later, IIRC)

To see non-abbreviated compilation commands, pass in V=1
as documented in the "Optional Features" section of the help output of
configure.

Signed-off-by: Bernhard Reutner-Fischer <rep.dot.nop@gmail.com>

configure: tweak checking for botan

If no botan-config was found we would end up doing:
checking for botan-config... no
/scratch/src/bind10/configure: line 16704: --libs: command not found
/scratch/src/bind10/configure: line 16705: --cflags: command not found

Instead, do not run botan-config if there was no botan-config found
as we will bail out gracefully later on if the botan headers could not
be found.

Signed-off-by: Bernhard Reutner-Fischer <rep.dot.nop@gmail.com>

configure: tweak quoting in CXX_TRY_FLAG

Add proper quotes to be on the safe side.
Remove superfluous whitespace in AC_LINK_IFELSE clause while at it.

Signed-off-by: Bernhard Reutner-Fischer <rep.dot.nop@gmail.com>

[1534] Implement the setting

But, for some reason, it doesn't really work. The tests fail with
strange error. To be talked about on ML.

[1613] also test very badly malformed query (and FORMERR counter)

[1613] fix rcode counting, and add additional tests for that

Merge ongoing #1593 into #1534

To minimize future conflicts, the #1593 (sockcreator cleanup) might have
severe impact on the code.

[1580] a suggested cleanup: rename addNSCOMAINProof to addNSCOMAINProofByNSEC
so it's clearer that it's NSEC specific (now that we have an NSEC3 version
of addNXDOMAINProof).  For consistency rename the NSEC3 version to
addNSCOMAINProofByNSEC3.  also a bit reorganized the code calling these methods
(mostly a matter of taste though), and adjusted tests to make them pass.

[1580] handled one missing case: run time collision due to having matching
NSEC3 for the wildcard.

[1580] updated addNSEC3NXDOMAINProof() so we don't check duplicate NSEC3 for
now, as we discussed in #1583.  Adjusted the nxdomainWithNSEC3Proof test
accordingly.

[1580] simplified the cases with unexpected NULL results of findNSEC3().
adjusted the expected exception for nxdomainWithBadNextNSEC3Proof accordingly.

[1580] corrected the comment about how to construct the corresponding
wildcard name (the previous one was seemingly copied from the NSEC
counterpart, which is not applicable here).

[1580] removed incorrect/redundant comments.  the first line is simply
incorrect.  the rest of them seem to be covered by the in-function comments.

[1580] Merge branch 'master' into trac1580 to reduce conflicts with results
of other NSEC3 tickets.

[1580] removed white space at EOL and in blank line.

[1593] suggested style cleanup: use the camel style for function names per guide

[1593] some trivial style fixes.  mostly to be aligned with the guideline.
also constify some variables, and remove unnecessary cast.

[1580] Add Bad NSEC3 next_proof test.

[1580] Add new comments to NSEC3 proves for NXDOMAIN,and fix the
construction of wildcard domain name.

[1613] fixed trivial typos in comments.

[master] Merge branch 'trac1585'

[1613] typo

[1638] updated comment wording

[1638] in parseNSEC3ParamText(), checked the encoded salt length first to
reject too large salt sooner.

[1593] Minor tidy up

[1613] per rcode statistics

[1593] Update comments in tests (+ minor style changes)

[1649] address review comments

construct name as next closer+closest encloser, and then invoke findNSEC3 to confirm its non-exist

[1593] Replace macros in test code with temmplates/overloaded functions

Changelog

Merge #1596

[1596] Missed a CmdCtl's spec file

[1638] editorial cleanup: removed a white space at EOL.

[1638] consolidated common logic of parsing text/wire NSEC3/NSEC3PARAM RDATA
into a shared helper module.

[1638] an NSEC3 bug fix: reject padded base32hex string for next hash.
also made a minor adjustment to compare()

[1638] add some more bad from-wire data tests.  no need to fix the main code.

[1638] unified NSEC3PARAM::toWire() methods, fixing a small bug for empty salt.

[1638] corrected "from wire" NSEC3PARAM constructor behaviors

[1638] supported NSEC3PARAM in gen_wiredata.py.

[1638] adjusted NSEC3PARAM testdata so it's compatible with corresponding NSEC3
test data.  some test cases were adjusted accordingly, too.

[1638] corrected salt comparison logic in NSEC3PARAM

[1638] corrected toText() output of NSEC3PARAM with empty salt

[1638] fixed various "from text" bugs in NSEC3PARAM

[1638] pre-work refactoring: extracted some NSEC3 tests into TYPED_TESTs
so that they will be shared with NSEC3PARAM.

[1585] Merge branch 'master' into trac1585 with fixing conflicts, mainly
to be more synchronized with merged #1582.

[1593] Miscellaneous cleanup of comments etc

[master] Merge branch 'trac1582'

[1649] one more check, and doc update

[1649] oops removed the if there

[1649] added a few docstrings

[1649] use the new helper functions in a few old places

[1649] remove superfluous / from identifier

[1649] add index check

[1649] add a test for the failure scenario

[master] [1596] Fix obvious typo in makefile

[1593] Restructure code into a more logical order

add RFC 5155  Section 7.2.6. the NSEC3 wildcard answer responses is added in this branch

[1596] Update spec files for everybody

Or, everybody who accepts a shutdown command with pid now. Stats needed
to be updated because it did call functions based on the name and spec
file somehow.

[1582] minor cleanups

[1582] comments and dox

[1596] Fix obvious typo in makefile

Merge branch #1596

Conflicts:
src/bin/resolver/main.cc

[1596] Remove unneeded checks

They'd get caught by an exception anyway.

[1596] Initialize in constructor

[1580] trivial editorial cleanups: removed white spaces at EOL, folded long lines

[1585] implemented unsigned delegation with NSEC3 case

[1580] Add NSEC3 process for NXDOMAIN.

[1583] one more cleanup: removed the wildcardEmptyWithNSEC3 test.  This case
is no different from the non-empty wildcard NXRRRSET in terms of query
processing.

[1583] added a test case of bogus findNSEC3 result for the wildcard NXRRSET case.
also made a non-related cleanup: removed resetting of nsec3_fake_ in the
fake findNSEC3() method.  This reset wasn't really necessary, and unless/until
we need that it'd be better to limit the place of tweaking such mutable
variables.

[1583] added a test case of run time collision for wildcard no data + NSEC3
case.

[1583] not directly related, intermediate update: reject null RRset in
Message::addRRset().

[1583] supported the normal case of wildcard no data response with NSEC3.
the older version of wildcardNxrrsetWithNSEC3 was removed.  also extended
some existing tests so it's easier to detect the cause when subsequent
updates introduce a regression.

[1583] unrelated editorial cleanups: folded long lines, corrected return code
of find() based on recent updates (there's no WILDCARD_NXRRSET, etc.)

[master] Merge branch 'trac1576' with fixing conflicts.

[master] chnagelog for #1641

[master] Merge branch 'trac1641'

[1641] added some detailed description for compareVectors used in NSEC3 implementation.

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

[master] temporarily disable part of test

see ticket #1668

[1582] NXRRset proof for DS query

[1641] editorial cleanup: removed unnecessary ','

[1641] editorial cleanup: removed a redundant blank line