Commit 7648495a authored by Per Cederqvist's avatar Per Cederqvist

Make it easier to change the file descriptor reserved for valgrind.

Don't use a file descriptor that interferes with the test suite.
Don't log simple reverse lookup errors for localhost, since they
interfere with the test suite and are fairly common and harmless.
parent f8aa2605
2003-08-07 Per Cederqvist <ceder@ceder.dyndns.org>
Make it easier to change the file descriptor reserved for
valgrind. Don't use a file descriptor that interferes with the
test suite.
* src/server/testsuite/config/unix.exp (PROTECTED_FDS): Added a
comment that refers to ../Makefile.am. Check that valgrind_fd is
set properly.
(lyskomd_host): This is now set in site.exp, not here.
(l2g_start): Use valgrind_fd instead of hardcoding the value.
(spawn_lyskomd): Ditto.
(dbck_run): Ditto.
* src/server/testsuite/Makefile.am (VALGRIND_FD): New constant,
set to 21. Added a comment that refers to PROTECTED_FDS in
unix.exp.
(site.exp): Set lyskomd_host to whatever 127.0.0.1 resolves to,
or 127.0.0.1 if it cannot be looked up at all. Set valgrind_fd to
the constant defined in VALGRIND_FD.
(valgrind.wrap): Use VALGRIND_FD, instead of hardcoding the value
in two places. Changed the valgrind file descriptor to 21, which
is currently the highest reserved file descriptor.
Don't log simple reverse lookup errors for localhost, since they
interfere with the test suite and are fairly common and harmless.
* src/server/connections.c (dns_resolution): Don't log errors if
the lookup of localhost fails due to "nxdomain" or "inconsistent".
The AIX 4.2 and 4.3 builds have failed the tests for max open
clients since we enabled ADNS. Attempt to fix. But can they
really have an extra overhead of 8-9 file descriptors?
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment