If the configure was called with stricter flags (-Werror=unused-variable)
then configure script did not detect tread safe initialisation.
As a result of this client code was not build with mutexes.
conftest.c: In function 'main':
conftest.c:39:17: error: unused variable 'm' [-Werror=unused-variable]
pthread_mutex_t m = PTHREAD_MUTEX_INITIALIZER;
cc1: all warnings being treated as errors
configure:15331: $? = 1
configure:15338: WARNING: Pthread library not found! Clients will not be thread safe...
Simple patch is attached.
I've found an embarrassing logic failure in my Valgrind execution design in
CI: tests which were excluded from Valgrind checks weren't running at all,
instead of just running without Valgrind.
The first attached patch fixes that. The second patch makes CI handle test
executables which start with a dash. Not that we're likely to ever have any,
but just in case.
Sorry for overlooking this.