GitLab

We don't take any benefit from the parallel harness, since we have only
one test binary in each directory, and automake can't parallelize that.
But it was hiding test output, which was annoying.

Acked on the mailing list.

(reported by emhn: left over after a distclean)

Some of the makefiles clear that but not that one. We do
automate the distclean (via distcheck) as part of our
buildfarm. But I think this __pycache__ was introduced in python3.2 and
those build systems don't have it.

This is trivial so didn't get review.

The run.sh.in is in the distfile.
(issue reported by emhn: was removed with distclean but was in tarball)
Trivial so didn't get review.

They were installed twice (to /etc and to /share, removed the latter)
Also, the generated .pem files were included in the tarball, while they should be generated during the build process

Second part of review as found in http://bind10.isc.org/ticket/1901#comment:11

According to the first part of the review in http://bind10.isc.org/ticket/1901#comment:11

committing at my discretion.

without this SunStudio will be confused about the namespace for the
declaration of internal Context below and refuse to compile.

confirmed it works, okayed on jabber.

(-rpath isn't actually checked in this macro)

Some dynamic linkers are wise enough (at least the Linux one) to correctly pick
needed shared libraries without hardcoded rpath. The --disable-rpath configure
switch allows distributors to avoid hardcoded rpath paths in libraries and
executables.
Signed-off-by: Adam Tkac <atkac@redhat.com>

Should be about working with the new names now

- renamed 'bind10' to 'b10-init'
- renamed related files as well
- left alone the 'run' script, as well as the directory name
- added new 'bind10' script that execs b10-init

note this commit only handles file names, and internal references have not been updated yet

On my home system I received error when running unit tests like:

Message DHCP4_SERVERID_LOADED server-id /home/reed/opt/bind10/var/b10-dhcp4-serverid has been loaded from file %2
assertion ""Excess logger placeholders still exist in message" == NULL" failed: file "../../../../src/lib/log/log_formatter.cc", line 69, function "void isc::log::checkExcessPlaceholders(std::string*, unsigned int)"

I am using --enable-logger-checks. The build farm also uses that.
I am not sure why this was not seen  on the build farm.

I added the missing value for dhcp4 and dhcp6 and the make check completed.

It had slight review on jabber.